Skip to content

[P2 docs] Refresh compiler-architecture target registry and capability evidence for Amp #755

Description

@ScriptedAlchemy

Documentation drift

Audited docsite against main 14c9822bc6c01d8ff2788454d0e1dc7861bc3816 while expanding #754.

website/docs/{en,zh}/guide/concepts/architecture.mdx still describes the pre-Amp target registry:

  • its built-in adapter table lists only claude, codex, cursor, and portable;
  • the prose says createDefaultRegistry registers portable, then codex, claude, and cursor;
  • it therefore omits the current amp adapter and its pinned amp-0.0.0-20260907001852-gf348fed.json capability evidence.

Current production registry/tests include Amp as a built-in target. The convention-first/capability pages in #754 already present all five targets correctly, so the architecture page should not contradict them.

Required correction

Update both locales together:

  • Add amp to the built-in adapter/planner table, naming adapters/amp.ts and the pinned capability JSON.
  • Update the registry-order/registration prose to include Amp. Do not imply order is author-facing target precedence; target selection is normalized/sorted for composite identity.
  • Explain Amp's architectural difference briefly: a generated directory PluginAPI factory under .amp/plugins/<name>/, API-registered callbacks/Skills, and skill-scoped MCP constraints. Link to the generated Hosts matrix instead of copying its detailed evidence.
  • Keep portable as the default target and keep the statement that plugin is not a target.
  • Link the new/updated target-capability guide from docs: complete convention-first guides, targets, and capability navigation (EN/ZH) #754 so architecture readers can leave compiler internals for the author-facing support map.
  • Remove stale historical wording that implies the generated Hosts matrix has only four built-in adapters.

Acceptance

Run the docsite locale-drift/link/anchor gates and search the built site for stale statements equivalent to “portable, codex, claude, and cursor” where the text is claiming the complete built-in registry. The generated hosts.md/events.md/notices.md remain authoritative for detailed capability states; do not hand-copy those matrices into architecture.

This is documentation only. It does not request another adapter, target registry, capability system, or changes to Amp runtime behavior.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions