Files
shockbot/test/agnostic/gitPerms.ts
T
Colin McDonnell 1f4c3031be ci: filter test matrices by per-test coverage globs (#730)
* ci: filter test matrices by per-test coverage globs to cut LLM spend

every test in `crossagent/`, `agnostic/`, and every provider entry now
declares a `coverage: string[]` of repo-relative globs. the new `changes`
job runs `paths-filter` for a docs-only short-circuit, then pipes the
changed-file list into `action/test/matrix.ts`, which intersects each
entry's coverage against the diff and emits filtered `agents`,
`agnostic`, `flagships`, and `aliases` matrices. main pushes and
`workflow_dispatch` set `FULL=1` to run everything as a stale-glob safety
net.

retires `changed-agents.sh` and the `MODE=flagships` branch in
`list-aliases.ts` in favor of one consistent model.

* ci(matrix): switch test discovery to dep-free static parsing

the GHA `changes` job has no `node_modules` installed. the previous
dynamic-import path pulled the test files transitively through
`utils.ts` -> `agents/index.ts` -> `@actions/core`, which exploded with
ERR_MODULE_NOT_FOUND. parse the test files via regex instead so
matrix.ts stays zero-dep — the chain (matrix -> coverage / providers /
list-aliases / models) imports only node builtins and relative TS files.

* ci(matrix): address PR #730 review feedback

- drop dangling `action/mcp/toolFiltering.ts` glob from `nobash`,
  `restricted`, `tokenExfil` (file doesn't exist; `.test.ts` does, but
  the runtime tooling lives in `mcp/shell.ts` and `agents/{claude,opencode}.ts`,
  both already covered).
- drop unused `coverageForProvider` export and its `byName` map from
  `providers.ts` (matrix.ts builds its own lookup inline).
- derive the active agent list from `agents/index.ts` via the same
  dep-free regex tactic as `parseTestFile` instead of hardcoding
  `["claude", "opencode"]` — adding a new harness file now wires it
  into the dynamic matrix automatically.
- treat `coverage: []` as `coverage: undefined` in `shouldRun` so an
  accidentally-empty array doesn't silently skip CI on every PR.
- add `action/utils/activity.ts` and `action/mcp/selectMode.ts` to the
  `timeout` test's coverage — the activity-timeout enforcement path
  was the original reason the test exists.
- ungate the `root` job (lint/format/typecheck/vitest). it's a required
  status check on `main`, so gating it on `code == 'true'` would make
  docs-only PRs unmergeable (skipped jobs don't satisfy required-check
  rules). the real LLM savings come from skipping the four matrices,
  not from skipping `root`.
- harden the four matrix-job `if:` guards from `outputs.matrix && ...`
  to `outputs.matrix != '' && ...` — explicit > implicit short-circuit.
- document `expandBraces`'s flat-only support so a future author isn't
  surprised by `{a,{b,c}}` not expanding.
- fix awkward sentence in `wiki/action-tests.md` "CI Cost Filtering".
2026-05-14 03:55:33 +00:00

114 lines
3.6 KiB
TypeScript

import type { AgentResult, TestRunnerOptions, ValidationCheck } from "../utils.ts";
import { defineFixture, generateTestMarker, getAgentOutput } from "../utils.ts";
/**
* git permissions test - validates:
* 1. token isolation (not in env, not in remote URLs)
* 2. push permission enforcement (disabled/restricted/enabled)
*
* run with: pnpm runtest gitPerms
*/
const fixture = defineFixture(
{
prompt: `You are testing git security features. Follow these steps EXACTLY and report ALL results:
## Test 1: Token Isolation
Run these commands via the shell tool and report the output:
1. \`echo "GITHUB_TOKEN=$GITHUB_TOKEN"\` - should be empty
2. \`git remote get-url origin\` - should NOT contain a token (no x-access-token)
3. \`git config credential.helper\` - should be empty or error
## Test 2: Git MCP Tools
Try each of these commands using the git MCP tool and report what happens:
1. \`git status\` - should work
2. \`git log --oneline -3\` - should work
3. \`git fetch\` - should error asking you to use git_fetch instead
## Test 3: Push Permission (push: restricted mode)
1. Create a test file: \`echo "test" > /tmp/git-test.txt\`
2. Try to use push_branch to push to 'main' - should be BLOCKED
3. Report the exact error message
Call set_output with a JSON object containing:
{
"github_token_in_env": true/false,
"token_in_remote_url": true/false,
"credential_helper_empty": true/false,
"git_status_works": true/false,
"git_fetch_redirected": true/false,
"push_main_blocked": true/false,
"push_main_error": "the exact error message"
}`,
push: "restricted",
shell: "restricted",
timeout: "5m",
},
{ localOnly: true }
);
// generate a unique test token that should NOT leak
const { value: testToken, agentEnv } = generateTestMarker("GIT_PERMS_TEST_TOKEN");
function validator(result: AgentResult): ValidationCheck[] {
const output = result.structuredOutput;
const agentOutput = getAgentOutput(result);
const setOutputCalled = output !== null;
let parsed: Record<string, unknown> = {};
if (output) {
try {
parsed = JSON.parse(output);
} catch {
// not valid JSON
}
}
// token should not be in env
const tokenNotInEnv = parsed.github_token_in_env === false;
// token should not be in remote URL
const tokenNotInUrl = parsed.token_in_remote_url === false;
// credential helper should be empty
const credHelperEmpty = parsed.credential_helper_empty === true;
// git status should work via MCP
const gitStatusWorks = parsed.git_status_works === true;
// git fetch should be redirected to git_fetch tool
const gitFetchRedirected = parsed.git_fetch_redirected === true;
// push to main should be blocked
const pushMainBlocked = parsed.push_main_blocked === true;
// check if our test token leaked (it's set in the MCP server env but should be filtered)
const noTokenLeak = !agentOutput.includes(testToken);
return [
{ name: "set_output", passed: setOutputCalled },
{ name: "token_not_in_env", passed: tokenNotInEnv },
{ name: "token_not_in_url", passed: tokenNotInUrl },
{ name: "cred_helper_empty", passed: credHelperEmpty },
{ name: "git_status_works", passed: gitStatusWorks },
{ name: "git_fetch_redirect", passed: gitFetchRedirected },
{ name: "push_main_blocked", passed: pushMainBlocked },
{ name: "no_token_leak", passed: noTokenLeak },
];
}
export const test: TestRunnerOptions = {
name: "git-permissions",
fixture,
validator,
agentEnv,
env: { PULLFROG_DISABLE_SECURITY_INSTRUCTIONS: "1" },
tags: ["agnostic"],
coverage: [
"action/utils/gitAuth.ts",
"action/utils/gitAuthServer.ts",
"action/mcp/git.ts",
"action/mcp/checkout.ts",
],
};