Skip to content

fix(permissions): restore /yolo off hint for default-source skip warning (CL-9385) - #1185

Open
TheGreatAxios wants to merge 4 commits into
cl-8986--auto-yolo-and-yolo-is-yolo-not-autofrom
cl-9385-restore-yolo-off-hint-for-default-source-skip-warning
Open

TheGreatAxios wants to merge 4 commits into
cl-8986--auto-yolo-and-yolo-is-yolo-not-autofrom
cl-9385-restore-yolo-off-hint-for-default-source-skip-warning

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Collaborator

Stacks on #1169 (owns the unified saved-skip warning); restores the in-session /yolo off hint for the default machine-wide settings source.

  • When the active settings source is the default machine-wide file, the warning appends (/yolo off to re-enable)
  • Custom-source warnings keep the file-path wording with no false provenance (byte-identical output)
  • Tests pin both variants: helper unit tests, TUI wiring tests, exec stderr test

Verification

  • RED: default-hint assertions failed on all three suites before the fix; custom pins passed
  • bun run typecheck: exit 0
  • bun run lint: exit 0
  • bun run test (bun test ./src ./tests ./evals ./scripts --randomize --seed 424242): 8035 pass, 0 fail, exit 0

Fixes CL-9385

@linear-code

linear-code Bot commented Sep 26, 2026

Copy link
Copy Markdown

CL-9385

Exec cannot run slash commands, so the recovery hint is TUI-only.
Default-source exec stays path-only; custom sources stay path-only.
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