Skip to content

Commit e019766

Browse files
os-zhuangclaude
andauthored
fix(spec): remove 8 dead page names from build-docs integration category (#8219)
`packages/spec/scripts/build-docs.ts`'s `integration` category listed 9 page names, 8 of which have never had a matching `.zod.ts` file in `packages/spec/src/integration/` since #4480 removed the per-provider connector template files. `buildCategoryPages` already filters by what was actually emitted, so the dead names were silently harmless — this removes them deliberately, matching the discipline the `automation` category's comments already established. `connector` is the only module `integration/` has ever emitted a page for. Verified with a fresh `gen:docs` run: 230 files before and after, byte-for- byte identical — the removal is inert by construction. Fixes #8166 Co-authored-by: Claude <noreply@anthropic.com>
1 parent 0704c98 commit e019766

1 file changed

Lines changed: 16 additions & 3 deletions

File tree

packages/spec/scripts/build-docs.ts

Lines changed: 16 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -501,9 +501,22 @@ const SECTION_GROUPS: Record<string, Array<{ section: string; pages: string[] }>
501501
// `connector-auth` removed at #4696 — `integration/` has no such file; the
502502
// five `ConnectorInstance*Auth` schemas reach this entry point through
503503
// `integration/connector.zod.ts`, and are documented there now.
504-
{ section: 'Connectors', pages: ['connector', 'mapping', 'translation'] },
505-
{ section: 'Transport & Storage', pages: ['http', 'message-queue', 'object-storage', 'offline'] },
506-
{ section: 'Tenancy', pages: ['tenant', 'misc'] },
504+
//
505+
// `mapping`, `translation`, `http`, `message-queue`, `object-storage`,
506+
// `offline`, `tenant` and `misc` never had a matching `.zod.ts` here
507+
// after #4480 removed the per-provider connector template files
508+
// (`connector/saas.zod.ts`, `connector/database.zod.ts`, file-storage,
509+
// message-queue, github, vercel) — the losing side of ADR-0023's
510+
// rejected per-system modelling; ADR-0097's answer is that provider
511+
// shapes come from the provider itself, not from the spec
512+
// (`integration/index.ts` records the six removed files). This
513+
// integration `message-queue` is that connector-template entry, not the
514+
// `system` category's own `message-queue.zod.ts` (#8075). `connector` is
515+
// the only module `integration/` has ever emitted a page for.
516+
// `buildCategoryPages` filters by what was emitted, so leaving these dead
517+
// names here would have been silently harmless — which is why they are
518+
// removed deliberately instead (#8166).
519+
{ section: 'Connectors', pages: ['connector'] },
507520
],
508521
kernel: [
509522
{ section: 'Plugin Lifecycle', pages: ['plugin', 'plugin-lifecycle-events', 'plugin-lifecycle-advanced', 'plugin-runtime', 'plugin-loading', 'plugin-registry', 'plugin-structure', 'plugin-validator'] },

0 commit comments

Comments
 (0)