Skip to content

Commit f7d5524

Browse files
committed
no-mistakes(document): Docs already covered the persona refactor thoroughly; fixed one inaccuracy: ABSENT vs UNREADABLE persona states were conflated in AGENTS.md and docs/configuration.md omitted the UNREADABLE case entirely, so both now describe it as its own distinct repair failure matching bin/fm-session-start.sh's actual behavior.
1 parent b7511f2 commit f7d5524

2 files changed

Lines changed: 3 additions & 1 deletion

File tree

AGENTS.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,8 @@ Tracked native session-open adapters only nudge this command; `docs/sessionstart
136136
Read the complete digest once and trust it as this turn's startup and recovery input.
137137
Do not separately re-read the persona, context, backlog, metadata, or bulk status inputs it just printed unless a source was reported absent or corrupt, older history is specifically needed, or a targeted workflow must inspect before writing.
138138
An `ABSENT` captain, shared-captain, secondmate, or learnings file means the firstmate repo's built-in defaults, no shared captain preferences, no registered secondmates, or no captured learnings; rebuild an absent or stale project registry from the clones before dispatch.
139-
An `ABSENT` persona is not a normal state, unlike those files: the tracked `persona.md` default should always exist, so its absence alongside an absent `config/persona.md` override means the tracked file was deleted or is unreadable and needs repair.
139+
An `ABSENT` persona is not a normal state, unlike those files: the tracked `persona.md` default should always exist, so its absence alongside an absent `config/persona.md` override means the tracked file was deleted and needs repair.
140+
A present but `UNREADABLE` persona file (a permissions problem, not a missing one) is reported as its own distinct repair failure rather than silently falling back or going unreported.
140141

141142
If the session lock cannot be acquired and verified, report its exact diagnostic and remain read-only; another active session is only one possible cause.
142143
A lock-refused session must not spawn, steer, merge, drain the wake queue, repair supervision, repair a checkout, or perform any other fleet mutation.

docs/configuration.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ The captain-facing voice - address term, seasoning policy, house vocabulary, and
1313
Swap the voice by editing tracked `persona.md` directly, or by dropping a local, gitignored `config/persona.md` in this home; when present, the local file fully replaces the tracked default rather than merging with it, mirroring `config/crew-harness`'s override pattern.
1414
`bin/fm-session-start.sh` resolves and prints the active persona's full contents unconditionally, every session, labeled by source (tracked default vs local override), so the voice is always in force with no per-reply trigger to load it.
1515
An absent persona (both tracked `persona.md` and `config/persona.md` missing) is reported distinctly from an `ABSENT` context-digest file: unlike `data/captain.md` or `data/learnings.md`, there is no built-in-defaults fallback, so it signals the tracked file needs repair.
16+
A present but unreadable active persona file (for example, a permissions problem) is reported as its own distinct `UNREADABLE` repair failure rather than silently falling back to the other candidate or going unreported.
1617

1718
## Operational home layout and state
1819

0 commit comments

Comments
 (0)