[orchestrator] Found by product-architect during a deliberate sweep of the wiki auth/config tables while fixing #1992. Filed rather than fixed there, to keep that issue scoped to the OIDC defect.
Problem
The Backup & Restore section of wiki/Architecture.md states two things that are both false:
BACKUP_DIR's default is documented as (none). In fact server/src/plugins/config.ts:259 reads getValue('BACKUP_DIR') ?? '/backups' — there is always a value.
- It states "Backup functionality is enabled when
BACKUP_DIR is set. If unset, all /api/backups/* endpoints return 503." Because the default is unconditional, backupEnabled = !!backupDir (config.ts:288) is always true, so the documented 503 path is unreachable from an operator's point of view.
CLAUDE.md already documents the /backups default correctly, so the wiki is the lone outlier.
Why it matters
This is the same defect class as #1992 — wiki prose describing a gate that the code does not implement — but it is a wrong default plus a dead gate rather than a phantom variable, in a different table. An operator reading this has no way to know backups are on by default, and may believe they have disabled the feature by leaving BACKUP_DIR unset.
Acceptance Criteria
Notes
[orchestrator] Found by
product-architectduring a deliberate sweep of the wiki auth/config tables while fixing #1992. Filed rather than fixed there, to keep that issue scoped to the OIDC defect.Problem
The Backup & Restore section of
wiki/Architecture.mdstates two things that are both false:BACKUP_DIR's default is documented as(none). In factserver/src/plugins/config.ts:259readsgetValue('BACKUP_DIR') ?? '/backups'— there is always a value.BACKUP_DIRis set. If unset, all/api/backups/*endpoints return 503." Because the default is unconditional,backupEnabled = !!backupDir(config.ts:288) is always true, so the documented 503 path is unreachable from an operator's point of view.CLAUDE.mdalready documents the/backupsdefault correctly, so the wiki is the lone outlier.Why it matters
This is the same defect class as #1992 — wiki prose describing a gate that the code does not implement — but it is a wrong default plus a dead gate rather than a phantom variable, in a different table. An operator reading this has no way to know backups are on by default, and may believe they have disabled the feature by leaving
BACKUP_DIRunset.Acceptance Criteria
wiki/Architecture.mddocumentsBACKUP_DIR's default as/backups, matchingconfig.ts:259andCLAUDE.md.backupEnabledis unconditionally true — or, if the 503 path is intended to be reachable, that is raised as a code defect in a separate issue rather than documented as if it works.Notes
product-architect(owns the Architecture wiki page).