[schmitch]

No issue asks for this. It follows #884: formats are crates linked into the server, not plugins, so the names now say so.

Renamed

Before	After
crates/plugin-api (forge-plugin-api)	crates/format-api (forge-format-api)
trait FormatPlugin	FormatSpec
crates/plugin (forge-plugin)	gone: folded into forge-core
forge_plugin::PluginFormat, UploadFormat	forge_core::format::SpecFormat, UploadFormat
forge_plugin::descriptor::check	forge_core::format::descriptor::check
server module plugins (plugins.rs)	formats (formats.rs)
registry plugin(), plugins(), with_plugin(s), images_plugin()	spec(), specs(), with_spec(s), images_spec()
tests/plugin_contract.rs	tests/format_contract.rs
"format plugin" in comments, docs and AGENTS.md	"format" or "format crate"

Folding the adapter into forge-core is safe: core already depended on the contract crate. Core's tests also no longer need the dev-dependency cycle through forge-plugin.

AGENTS.md's rule "no crates/format-<id>" now says that crates/format-api is the contract, not a format. docs/format-plugins.md also drops its last "Contract 0.4.0".

Changes visible outside the code

• A format's internal error answers format failed instead of format plugin failed. That text is not part of any client protocol, and no test asserts it.
• One warning log line, about a manifest summary that isn't a JSON object, names the format under format instead of plugin.

Kept on purpose

• The telemetry surfaces plugin_npm, plugin_raw and plugin_other, with plugin_surface and Surface::Plugin*. docs/deployment.md documents these metric values, and dashboards and alerts select on them; the doc comment now says why they keep the name.
• The file name docs/format-plugins.md, now titled "Format crates". CHANGELOG.md links to it and to its #files-formats and #images-formats anchors, and changes never edit CHANGELOG.md.
• Other projects' plugins: nuget-plugin-forge (NuGet's credential provider), the config test of the removed [plugins] section, Vite's and Verdaccio's plugins, and the CSP's "no plugins".

Checks

Rebased on main at 388fb93, after #881, #895, #897 and #890:

• the names those added (summary_caps, FilePackage, the streamed index building) are renamed too;

• Remove unused code and dependencies #890's dependency removals are kept.

• cargo fmt --all --check passes.

• cargo clippy --workspace --all-targets -- -D warnings passes; it builds every integration test too.

• Unit tests pass with --locked:

  • forge-format-api, the five format crates and forge-telemetry: 317 passed;
  • forge-core --lib: 373 passed;
  • forge-server --lib: 291 passed;
  • format_contract::the_probe_is_a_valid_format: passed.

• Web: tsc --noEmit passes, and Vitest passes (625 tests).

• cargo xtask docs --check and cargo xtask changelog --check pass.

• Not run locally: the server and core integration tests, which need Postgres and S3. CI runs them, and they were green on the previous head.

🤖 Generated with Claude Code

https://claude.ai/code/session_012e6DJXVot5LKkJJWpxY1bT