diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md index a071079..c79a764 100644 --- a/.github/pull_request_template.md +++ b/.github/pull_request_template.md @@ -7,7 +7,7 @@ A public patch uses Node.js 20 and the Node suites that match the change. You do not need the agent harness or a task session directory. -- [ ] `npm run test:broker-core` / `npm run test:client` / `npm run test:harness-adoption` as applicable, or I explained why a deterministic test is not available +- [ ] `npm run test:broker-core` / `npm run test:client` / `npm run test:harness-adoption` / `npm run test:docs` as applicable, or I explained why a deterministic test is not available - [ ] I did not commit private paths, credentials, generated Xcode projects, local broker state, or task-session artifacts ## Notes diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 987f944..20b2c08 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -38,8 +38,9 @@ npm run test:harness-adoption Run `npm run test:docs` for doc, README, or template changes; it runs the `docs/test` front-door checks. App work also needs XcodeGen and `npm run test:app`. The full suite is -`npm test`. The same Node suites run on GitHub-hosted Ubuntu CI. That job -does not run `npm run test:app`. +`npm test`. GitHub-hosted Ubuntu CI runs public-surface, `test:broker-core`, +`test:client`, and `test:harness-adoption`. That job does not run +`npm run test:app` or `npm run test:docs`. You do not need to run `agent:context`, `agent:verify`, or `agent:complete`, and you do not need to create a task session directory. diff --git a/docs/test/front-door.test.mjs b/docs/test/front-door.test.mjs index 09b78ed..59bd592 100644 --- a/docs/test/front-door.test.mjs +++ b/docs/test/front-door.test.mjs @@ -138,6 +138,11 @@ test("CONTRIBUTING leads with a public Node-20 patch track that does not require || human.includes("npm test"), "public track must name a Node test command that is not agent:*", ); + assert.ok(human.includes("npm run test:docs"), "public track must name test:docs"); + assert.ok( + human.includes("does not run") && human.includes("npm run test:docs"), + "public track must not claim Ubuntu CI runs test:docs", + ); assert.equal(human.includes("You do not need to run `agent:context`"), true); assert.equal(human.includes("$HOME/.codex"), false); assert.equal(human.includes("npm run agent:context --"), false); @@ -277,6 +282,7 @@ test("PR template is a public-patch checklist and does not require the harness", assert.ok(template.includes("npm run test:broker-core")); assert.ok(template.includes("npm run test:client")); assert.ok(template.includes("npm run test:harness-adoption")); + assert.ok(template.includes("npm run test:docs")); assert.ok(template.includes("Alpha")); assert.ok(template.includes("macOS")); assert.ok(template.includes("Xcode"));