Skip to content

soldeerignore .gas-snapshot: the sensitive-file prompt kills headless publishes - #60

Merged
thedavidmeister merged 1 commit into
mainfrom
2026-08-20-soldeerignore-gas-snapshot
Aug 20, 2026
Merged

soldeerignore .gas-snapshot: the sensitive-file prompt kills headless publishes#60
thedavidmeister merged 1 commit into
mainfrom
2026-08-20-soldeerignore-gas-snapshot

Conversation

@thedavidmeister

@thedavidmeister thedavidmeister commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Package Release on main has failed twice since PR #59 merged, both with soldeer's error during IO operation for "": not connected immediately after "You are about to include some sensitive files in this version". That is the interactive sensitive-file prompt dying on closed stdin in headless CI — the exact failure the org's roh SKILL.md warns about. The trigger is .gas-snapshot, introduced by #59 and not in this repo's .soldeerignore (which predates the file); rain.extrospection's ignore file already excludes it, and its publishes are healthy.

One line: .gas-snapshot added to .soldeerignore, matching the extrospection precedent. Verified by forge soldeer push --dry-run: the zip now carries src/, test/, LICENSE(S) and README only. Merging this retriggers Package Release on the push, which should publish the blocked rain-factory 0.1.9 (PR #58+#59 content).

QA

  • Discriminating tests: n/a - one-line publish-config change; the discriminator is the Package Release run on the merge push succeeding where the last two failed identically, plus the local dry-run zip listing (before: .gas-snapshot included; after: absent)
  • Mutations applied: n/a - no code; a mutated ignore entry either re-includes the dotfile (reproducing the observed CI failure) or excludes a published file (visible in the dry-run listing)
  • Oracle: rain.extrospection's .soldeerignore (the org precedent that publishes cleanly with a .gas-snapshot present) and the failing runs' own logs; the dry-run zip listing is independent of the change's author
  • Category check: unblocks "make rain.factory.deploy use rain.factory properly" - the deploy repo cannot pin the lib until the registry carries it; covered: the single file gating the publish

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated ignored files configuration to exclude gas snapshot artifacts.

…le prompt, which hangs headless publishes

PR #59 introduced .gas-snapshot; soldeer's sensitive-file prompt fired on
it during the main publish and died on closed stdin ('not connected'),
failing Package Release twice. rain.extrospection's .soldeerignore already
excludes it; aligned. Verified by dry-run: the zip now carries src/,
test/, LICENSE(S), README only.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Aug 20, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 5bde5999-fa5e-4f8b-a7d3-16c2658b808d

📥 Commits

Reviewing files that changed from the base of the PR and between bde568e and 3095b07.

📒 Files selected for processing (1)
  • .soldeerignore

Walkthrough

The Soldeer ignore list now excludes .gas-snapshot files.

Changes

Ignore list update

Layer / File(s) Summary
Configure Soldeer ignore rules
.soldeerignore
Added .gas-snapshot to the ignored files list.

Estimated code review effort: 1 (Trivial) | ~2 minutes

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch 2026-08-20-soldeerignore-gas-snapshot

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@thedavidmeister
thedavidmeister merged commit c1c2afd into main Aug 20, 2026
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant