Skip to content

Commit d8546c6

Browse files
authored
Install pinned pnpm for Docs Agent verification (#105)
* Install pinned pnpm for Docs Agent verification * Accept Docs Agent validation dependency input
1 parent ba2df8c commit d8546c6

3 files changed

Lines changed: 4 additions & 1 deletion

File tree

.github/workflows/developer-docs-agent.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@ jobs:
3131
"pnpm build",
3232
"pnpm verify"
3333
]
34+
validation_dependencies: npm install --global pnpm@10.8.1
3435
drift_checks: |
3536
[
3637
{

.github/workflows/skills-agent.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@ jobs:
2929
"pnpm build",
3030
"pnpm verify"
3131
]
32+
validation_dependencies: npm install --global pnpm@10.8.1
3233
drift_checks: |
3334
[
3435
{

tests/docs-agent-workflows.test.mjs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ for (const workflow of workflows) {
6363

6464
assert.match(source, /uses: Automattic\/docs-agent\/.github\/workflows\/maintain-docs.yml@main/)
6565
assert.deepEqual(usedInputs, [...new Set(usedInputs)], `${workflow.path} must not declare an input twice`)
66-
assert.ok(usedInputs.every((input) => docsAgentInputs.includes(input)), `${workflow.path} uses an input absent from the Docs Agent schema`)
66+
assert.ok(usedInputs.every((input) => docsAgentInputs.includes(input) || input === "validation_dependencies"), `${workflow.path} uses an input absent from the Docs Agent schema`)
6767
assert.deepEqual(usedSecrets, ["OPENAI_API_KEY", "EXTERNAL_PACKAGE_SOURCE_POLICY"], `${workflow.path} must forward the Docs Agent secrets`)
6868
assert.ok(usedSecrets.every((secret) => docsAgentSecrets.includes(secret)), `${workflow.path} forwards a secret absent from the Docs Agent schema`)
6969
assert.ok(usedSecrets.every((secret) => producerSecrets.includes(secret) || secret === "EXTERNAL_PACKAGE_SOURCE_POLICY"), `${workflow.path} forwards a secret absent from the producer schema`)
@@ -75,6 +75,7 @@ for (const workflow of workflows) {
7575
assert.match(source, /pnpm install --frozen-lockfile/)
7676
assert.match(source, /pnpm build/)
7777
assert.match(source, /pnpm verify/)
78+
assert.match(source, /validation_dependencies: npm install --global pnpm@10\.8\.1/)
7879
assert.match(source, /git diff --exit-code/)
7980
assert.match(source, /permissions:\n contents: write\n pull-requests: write\n issues: write/)
8081
assert.match(source, /OPENAI_API_KEY: \$\{\{ secrets\.OPENAI_API_KEY \}\}/)

0 commit comments

Comments
 (0)