fix: remove the .mode override; mode selection falls through to inference#72
Merged
Conversation
…ence .mode was gitignored, so PR #34's removal of agent mode could not reach it. A stale sentinel kept winning the selection chain and resolving to a modes/agent.md that no longer exists, so inference never ran. Durable mode preference now lives only in profile.md, which is read and announced each session. AGENTS.md now also states that an unrecognised mode name falls through to inference instead of being improvised on. That rule is the load-bearing half: deleting the local .mode fixes one clone, but other clones have their own gitignored copy that no commit can reach. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Snapshot of in-progress paper.md edits, carried along on this branch rather
than left dirty in the working tree. Incomplete: two placeholder stubs
("The first are the wiki pages...", "The harness."), the Benchmark examples
and Software design sections removed pending rewrite, an unresolved
affiliation note, and typos. Retitles PyAutoLens as PyAutoLens-JAX in the
summary against the existing Nightingale2021 citation.
Unrelated to the .mode fix in the parent commit; drop this commit if the
branch is merged before the rewrite lands.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The bug
.modeat the repo root is gitignored. PR #34 (e124ab1, Jul 8) removed agent mode and every reference to it — but it physically could not reach a gitignored file, and nothing validates one.So a sentinel written in June, naming a mode that no longer exists, kept winning the selection chain:
.modesaidagent,modes/agent.mdwas deleted, andAGENTS.mdsaid nothing about what to do with an unrecognised value. The assistant improvised. Inference never got a turn — which is what surfaced as "it keeps picking random modes".Nothing in code reads
.mode(no hook, no script,autoassistant/doesn't touch it), so this is docs-only.The fix
Chain is now: explicit instruction →
profile.md"Interaction mode" → infer from the opening request (fall back to assistant);.maintaineroutranks both.profile.mdkeeps the durable-override job because it's read and announced every session, so it can't silently rot the way an invisible sentinel did.AGENTS.mdalso now states that an unrecognised mode name is not a mode: ignore it, say so in one line, fall through to inference. That rule is the load-bearing half — deleting the local.modefixes this clone, but other clones still have their own gitignored copy that no commit can reach. With the fall-through rule, a staleagentthere degrades to a one-line note instead of undefined behaviour.Also removed the misleading
.modedocs inREADME.md, the.gitignoreentry, and a.modemention inmodes/maintainer.md. The profile template needed no change — it already lists onlyteacher/assistantand says to leave it unrecorded for inference.5367918carries in-progresspaper/paper.mdedits that were sitting dirty in the working tree, unrelated to this fix. It is not ready:"The first are the wiki pages, which provide the underlying"→"skills, autolens worksapce.", and a bare"The harness."# Update to My Newcastle affiliation)@Nightingale2021citationwhilest,worksapce, a line break splittingconversationalDrop
5367918before merge unless the rewrite lands first. The.modefix is the parent commit and is independent of it.🤖 Generated with Claude Code