Skip to content

docs(openspec): remove the stale AGENTS_FALLBACK requirement - #515

Merged
theCodeDrift merged 1 commit into
mainfrom
openspec/remove-agents-fallback-requirement
Oct 16, 2026
Merged

theCodeDrift merged 1 commit into
mainfrom
openspec/remove-agents-fallback-requirement

Conversation

@theCodeDrift

Copy link
Copy Markdown
Member

Removes the cli-init requirement "Codex install destination overrides the fallback for the same directory". This is a spec-only change, with no code or release note.

What it claimed

  • An AGENTS_FALLBACK tool descriptor existed alongside Codex, both with installDir: ".agents".
  • A "state-based cleanup helper" looked up tool descriptors by installDir and had to return Codex rather than AGENTS_FALLBACK.
  • The install summary named "Codex" for .agents/skills/ writes when .codex/ was present.

Evidence it is stale

  • grep -rn "AGENTS_FALLBACK" packages/cli/src finds nothing.
  • TOOLS in packages/cli/src/install/install.ts is detection-only. The install destination comes from the fixed SHIM_TARGETS catalog, which has a single .agents row (label: "Agent Skills", generic: true). DEFAULT_SHIM_DIR = ".agents" is what detectSelectedDirectories returns when nothing is detected.
  • packages/cli/src/install/state.ts has no tool-descriptor lookup at all. It diffs manifest targets by directory name.
  • The summary-labelling scenario contradicts "Wizard prompts the user to choose install locations", which says the .agents/ target SHALL be reported in the install summary as Agent Skills. The code follows that requirement.

Was any scenario still live?

None is lost:

Removed scenario Status
Codex detection labels the .agents/ install as Codex Contradicted by the wizard requirement. Codex is now named on the picker row, which "Codex is named in the tool list" specifies
Lookup by installDir resolves to Codex over fallback The helper and descriptor do not exist
Fallback still resolvable for legacy state without Codex detection Covered by "Agents fallback install" (".agents stub written when no tools detected") and by the wizard's "Agents is the default when nothing is detected or installed" and "Manifest-recorded locations are pre-checked"

Scenario diff

I diffed the sorted #### Scenario lines of openspec/specs/cli-init/spec.md before and after openspec archive. Only the three scenarios above disappeared, and nothing was added or changed:

< #### Scenario: Codex detection labels the .agents/ install as Codex
< #### Scenario: Fallback still resolvable for legacy state without Codex detection
< #### Scenario: Lookup by installDir resolves to Codex over fallback

The change is archived on this PR as openspec/changes/archive/2026-10-15-remove-agents-fallback-requirement/. pnpm openspec validate --strict passed before archiving.

@theCodeDrift theCodeDrift added the skip-changeset PR intentionally ships no release note (bypasses the changeset requirement) label Oct 16, 2026
@theCodeDrift
theCodeDrift merged commit 0307141 into main Oct 16, 2026
8 checks passed
@theCodeDrift
theCodeDrift deleted the openspec/remove-agents-fallback-requirement branch October 16, 2026 01:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changeset PR intentionally ships no release note (bypasses the changeset requirement)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant