Skip to content

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

Merged
TheGreatAxios merged 4 commits into
mainfrom
cl-9385-restore-yolo-off-hint-for-default-source-skip-warning
Sep 27, 2026
Merged

TheGreatAxios merged 4 commits into
mainfrom
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

@TheGreatAxios
TheGreatAxios deleted the branch main September 27, 2026 22:26
@TheGreatAxios TheGreatAxios reopened this Sep 27, 2026
@TheGreatAxios
TheGreatAxios changed the base branch from cl-8986--auto-yolo-and-yolo-is-yolo-not-auto to main September 27, 2026 22:29
@TheGreatAxios
TheGreatAxios force-pushed the cl-9385-restore-yolo-off-hint-for-default-source-skip-warning branch from 2073501 to 920d2c5 Compare September 27, 2026 22:33
@TheGreatAxios
TheGreatAxios enabled auto-merge (squash) September 27, 2026 22:33
@TheGreatAxios
TheGreatAxios merged commit 8494ae2 into main Sep 27, 2026
12 of 13 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