skillsaw testing.instructions.md

Test-writing guidance for checking a command-line tool from the outside, using realistic project folders and test data.

In plain words
What is it for?
Use it to add or update integration tests for the lint and fix commands, including tests with sample repositories, custom rules, and Unicode output.
Why use it?
It helps you choose between fast in-process tests and real subprocess tests when only a separate process can prove behavior such as imports or text encoding.

Instructions file for GitHub Copilot

Install

Getting it into your agent

One page per mod, every tool's command on it. A separate URL per tool would split the same page into five that compete with each other.

agentmods
npx agentmods add instructions/stbenjam/skillsaw/testing
Clone the repo
git clone --depth 1 https://github.com/stbenjam/skillsaw

Made for: GitHub Copilot.

Per session 661 This file is loaded in full into every session.
When invoked 661 The same file — it is already loaded in full.
Security scan A 1 finding. Scan, not verified.
Origin original No closer match found in the catalogue.
Token cost

What it costs to keep this loaded

Counted locally with the o200k_base tokenizer, which is exact for GPT models; Claude uses its own tokenizer and its counts differ. Treat this as one consistent yardstick across the catalogue rather than a bill. Prices are per million input tokens.

ModelPer sessionOnce invoked
Fable 5 $0.00661 $0.00661
Opus 5 $0.00331 $0.00331
Sonnet 5 $0.00132 $0.00132
Haiku 4.5 $0.00066 $0.00066

Measured 2d ago against content hash 2b2164b556f5, method: parsed. Prices are Anthropic first-party input rates as of 2026-08-30, from the pricing page.

Security

Grade A, and why

skillsaw testing.instructions.md scanned grade A with 1 finding against 26 rules in 11 categories — prompt injection, anti-refusal, data exfiltration, privilege escalation, supply chain, agent snooping, system-prompt leakage, SSRF and excessive agency — measured 2d ago.

A static scan of the body, not an audit. Every finding is printed with the line that produced it so you can judge whether it matters here. A mod is markdown that instructs an agent; that is exactly why what it instructs is worth reading.

Runs shell commandslowCapability

Expected in a hook, worth knowing in a rule or an instructions file.

`subprocess.CompletedProcess` stand-in. Spawning an interpreter per call cost
.github/instructions/testing.instructions.md · 65 lines

How it starts

The opening of the file, as written. The whole thing — 65 lines — stays where its author put it; the contents beside it link to each section on GitHub.

Testing

Integration Tests

Integration tests live in tests/test_integration.py and verify end-to-end behavior by driving the CLI (skillsaw lint / fix) against realistic repo fixtures.

Drive the CLI through run_cli

run_lint and the _run_fix helpers call run_cli from tests/cli_runner.py, which invokes skillsaw.cli.main() in-process with argv, stdout, stderr, env and cwd swapped, and returns a subprocess.CompletedProcess stand-in. Spawning an interpreter per call cost ~160ms — more than half of it importing skillsaw — and dominated the suite. Prefer run_cli for a new integration test.

Reach for a real subprocess.run only when the test depends on something a shared interpreter cannot give it:

  • Import isolation — the --no-custom-rules tests prove a rule file on disk did or did not get imported. In-process they would execute an arbitrary rule inside the test worker, and sys.modules could mask a repeat import and make the negative case pass vacuously.
  • A real encoding on stdout — the lone-surrogate regressions assert that rendering a report raises no UnicodeEncodeError. A StringIO never encodes, so in-process they would pass vacuously.
  • TTY and colour-cascade behavior, which depends on the real stream.
  • Entry-point discoverytests/test_integration_plugins.py installs plugins and needs a fresh interpreter to pick them up, so it stays on subprocess throughout.

Anything run_cli runs shares one interpreter, so a helper that leaves process-global state behind breaks later tests. run_cli already resets the two that outlive a call — root logging handlers and the skillsaw.utils file cache; add to _reset_process_globals if you introduce another.

Use test fixtures

Prefer static fixtures in tests/fixtures/ over building repos programmatically in test code. Fixtures should contain realistic file content that mirrors what real users would write.

Read the full file on GitHub · 65 lines

Changes

What this file has done since we first saw it

Hashed on every crawl. A supply-chain change to an agent config is a question of when, not whether, so the history is kept rather than the latest state alone.

  1. 2d ago First seen · 65 lines · 661 tokens per session scan A 2b2164b556f5

Subscribe to this mod's changes

skillsaw testing.instructions.md is an instructions file published in the GitHub repository stbenjam/skillsaw (59 stars, last pushed 2d ago), licensed Apache-2.0. It adds 661 tokens to every session, about $0.0033 per session on Opus 5. A static security scan graded it A with 1 finding (runs shell commands). No closer match exists in the catalogue, so it is treated as the original; first seen 2026-08-30.

Related

Other instructions, from other repositories

codex AGENTS.md

AGENTS.md instructions for openai/codex, covering rust/codex-rs, the codex-core crate, code review rules, crate api surface and model visible context.

openai/codex · 5,182 tokens

buildNext

Working notes and architecture documentation for the new esbuild-based build system in build/next. Use when making changes to the new build pipeline (transpile/bundle commands, NLS plugin, source-map handling, resource copying, or self-hosting watch tasks).

microsoft/vscode · 6,785 tokens

next.js AGENTS.md

Instructions for vercel/next.js, covering next.js development guide, codebase structure, monorepo overview, core package: packages/next and other important packages.

vercel/next.js · 7,296 tokens

vscode oss-third-party-notices.instructions.md

Instructions for microsoft/vscode, covering vs code oss third-party-notices pipeline, architecture, pipeline flow in ci, applying the notice (cutover) and fallback chain (never fail the build).

microsoft/vscode · 5,001 tokens

spec-kit AGENTS.md

Instructions for github/spec-kit, covering agents.md, about spec kit and specify, quickstart — add a new integration in 5 steps, integration architecture and integrationmanifest — file tracking.

github/spec-kit · 7,040 tokens

langchain AGENTS.md

Instructions for langchain-ai/langchain, covering global development guidelines for the langchain monorepo, corridor security analysis, project architecture and context, monorepo structure and development tools & commands.

langchain-ai/langchain · 4,345 tokens