Skip to content

prove: pick Yes on the folder trust dialog - #17

Open
karanb192 wants to merge 1 commit into
mainfrom
trust-dialog-fix
Open

karanb192 wants to merge 1 commit into
mainfrom
trust-dialog-fix

Conversation

@karanb192

Copy link
Copy Markdown
Owner

Summary

  • scripts/prove.mjs: the first-run answer for the folder trust dialog is now Down then Enter. On Claude Code 2.1.288 the dialog highlights "No, exit" first, so a bare Enter exited the session and the stage failed with first-run screen "trust" did not go away.

Details

Observed on screen in an interactive run on 2.1.288 (the previous comment said the string had not been seen on screen yet). The comment on the table entry now records the observation.

Two more observations from the same runs, not changed in this PR, recorded for a follow-up:

  • The interactive stage's run directory sits under the harness home in ~/.cache, so Claude Code's memory walk-up loads the person's own ~/.claude/CLAUDE.md as a project file into the session, and the isolation stage then fails naming the real config. A run directory outside the home directory (for example under /private/tmp) does not load it.
  • A type step sent right after the prompt marker appeared did not reach the input box in one run (the stream never carried the text); a separate manual tmux test on the same build showed that Escape does not clear typed text. The cause is not pinned down yet.

Test plan

  • node --test 'plugins/mod-builder/skills/mod-builder/tests/*.test.mjs' passes
  • An interactive prove run on 2.1.288 gets past the trust dialog (observed on this machine after the change)

This branch has not been deployed

No deployments
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