chore(detent): migrate workflow layout#251
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: de252a7042
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| @@ -0,0 +1,177 @@ | |||
| schema: 1 | |||
There was a problem hiding this comment.
With the split layout enabled by this new detent.yaml, Detent now resolves optional machine-local overlays named detent.local.yaml and WORKFLOW.local.md beside the workflow definition. This migration leaves those files trackable, so the next operator who creates local paths or credentials in an overlay will see them as normal untracked files and can accidentally commit them; Detent's workflow-layout migration docs call out adding both overlay names to .gitignore as part of the split migration.
Useful? React with 👍 / 👎.
1 participant
Summary\n\n- Move Detent runtime configuration out of WORKFLOW.md into root detent.yaml using the v0.48.0 migration.\n- Preserve the workflow prompt and effective runtime behavior while adopting the canonical split layout.\n\n## Test Plan\n\n- Run the workflow-layout migration again in dry-run mode and confirm it is a no-op.\n- Run detent doctor for gopher-ai and confirm the split revision matches the live runtime.\n- Run the shared-sync, installation, command, hook, worktree, and ship gate test suites.