agentleFS
shinprshinpr/codex-workflows/.agents/skills/ai-development-guide/SKILL.md

ai-development-guide

Skill39 starsChanged 10 days agoMITView on GitHub

Root-cause discipline, proportionate impact analysis, and implementation completeness assurance. Use when fixing bugs, reviewing code quality, refactoring, making technical decisions, or performing quality assurance.

2 other projects use a version of this.

Adopted by

Most recently taken up by

The file

We split this skill into the rules it gives an agent

7 rules. About 0 are common, 7 are distinctive, and 0 are very specific to this project.

  • ≈0

    Common · estimate

    Many files like this have a rule of this kind.

  • ≈7

    Distinctive · estimate

    This author's own choices. Worth a look.

  • 0

    Very specific

    Tied to their project's names, files or tools. Probably skip.

  • Common
  • Distinctive
  • Very specific
  1. Distinctive:

    Read references/frontend.md only for React or TypeScript frontend work whose changed behavior or quality failure needs those rules.

    line 10
  2. Distinctive:

    When an unverified premise could invalidate the approach, check it by the cheapest sufficient means before expanding dependent work.

    line 16
  3. Distinctive:

    When an observed failure exists:

    line 31
  4. Distinctive:

    Verify the original failure and the affected contract.

    line 36
  5. Distinctive:

    Before changing code, inspect the target and enough representative callers, consumers, tests, configuration, and siblings to determine:

    line 42
  6. Distinctive:

    Prefer the lowest-total-complexity implementation that satisfies the current outcome.

    line 56
  7. Distinctive:

    Keep similar local code separate when its responsibilities may evolve independently or abstraction adds more contract surface than it …

    line 58

≈ “Common” is an estimate: at least 1 in 5 of the 350,495 skill files we read have a rule of the same kind, not the same words. “Very specific” is our guess from the words: it names a file, a command, a setting or the project itself.

Keep what fits and drop the rest. Your version, ready for your agents.
Files
SKILL.md

AI Development Guide

Reference

Read references/frontend.md only for React or TypeScript frontend work whose changed behavior or quality failure needs those rules.

Outcome Boundary

Ground the required outcome in the governing task or design sources and the user's explicit instructions. Choose work by what the consumer must be able to do or judge, preserving that result while reducing the means needed to deliver it.

When an unverified premise could invalidate the approach, check it by the cheapest sufficient means before expanding dependent work. When results contradict that premise, reconsider the approach before patching its parts. Keep this reasoning in the current task; a small request needs no additional artifact.

Phase passage permits progress; technical necessity remains a separate evidence-backed lifecycle decision. Reconsider and remove or supersede means recorded in a Design Doc, ADR, plan, AC, or test when the required outcome remains satisfied. Preserve user-required outcomes, explicit constraints, actual consumer contracts, and external-action authority. For authorized feature removal, check remaining consumers; the removed entry point and its now-unused backend create no preservation obligation. Coordinate affected source and consumer updates through the orchestrator; existing execution authority continues for outcome-preserving technical reductions.

Investigate, repair, refactor, and verify only as far as one of these requires:

  • a user-required outcome or explicit constraint;
  • a dependency needed for that outcome;
  • an observed failure or contradiction in the changed path;
  • an evidence-backed material risk created or exposed by the change.

Evidence makes a candidate evaluable. Adopt work only when it is necessary now and its benefit justifies total lifecycle cost over no change, subtraction, or reuse. A required outcome defect still needs a sufficient response. Keep unrelated debt outside the change.

Root-Cause Discipline

When an observed failure exists:

  1. Reproduce or identify the failing observable condition.
  2. Trace the responsible control, data, or state path until the cause is supported by evidence.
  3. Correct the cause at the smallest responsibility boundary that preserves the governing contract.
  4. Verify the original failure and the affected contract.

Stop causal questioning when the evidence supports the responsible path and its verification. Apply a direct correction when the cause and proof are already evident. Keep root-cause reasoning in the active task or response; create a separate artifact only for a named downstream consumer. Preserve error visibility and test strength, and correct the observed cause rather than masking it with an unconditional fallback or symptom patch.

Proportionate Impact Analysis

Before changing code, inspect the target and enough representative callers, consumers, tests, configuration, and siblings to determine:

  • the contract being changed or preserved;
  • the directly affected responsibility and dependency direction;
  • the observable verification that can prove the outcome;
  • any adjacent file required for the same outcome.

When the change alters a public, shared, serialized, or persistent contract and its consumers are enumerable, account for every known consumer. For other changes, representative inspection is sufficient. Stop expanding the search when additional context cannot change the implementation or verification decision. Record findings in the active task or response only when another worker needs them.

For an observed bug or regression, inspect adjacent cases that share its supported cause, contract, or state boundary. Include an adjacent case in the change only when leaving it unchanged would keep the same in-scope failure active.

Design and Reuse Judgment

  • Treat complexity as cost, never as evidence of value. Additional user decisions, settings, modes, concepts, outputs, persistent state, and implementation paths earn their cost only when a current need in the requirements cannot be served by a smaller existing or default behavior and the observable benefit justifies the total UX and lifecycle burden.
  • Prefer the lowest-total-complexity implementation that satisfies the current outcome.
  • Reuse or extend an existing element when it owns the same responsibility and represents the repository’s current pattern.
  • Keep similar local code separate when its responsibilities may evolve independently or abstraction adds more contract surface than it removes.
  • Introduce shared state, public fields, modes, flags, fallbacks, abstractions, services, or dependencies only when current evidence requires them.
  • Resolve technical reductions within the authorized outcome and existing execution authority, updating affected design assumptions and proof. Return changes to user-required outcomes, explicit constraints, actual consumer obligations, or irreversible-action authority to the orchestrator for user approval.

Error and Fallback Safety

Preserve useful error context and keep failures observable. Limit fallbacks to degraded behavior defined by a requirement, accepted design, or representative repository contract. Limit logging, metrics, and operational machinery to those supported by the current outcome or representative repository practice.

Quality Assurance

Discover applicable checks from the changed file types, task verification methods, project manifests, configuration, and CI. Run:

  1. the focused check that observes the changed behavior or artifact;
  2. static analysis, formatting, build, unit, integration, or E2E commands that the repository or governing task requires for this change;
  3. any wider check needed because the change crosses that boundary.

Limit checks, thresholds, environments, external connections, and test lanes to those required by the repository or governing artifact. Reuse valid task-specific evidence; rerun it after a later fix that can invalidate it.

Fix failures caused by the current change and failures within required dependencies. Report unrelated baseline failures with evidence; they block completion only when they prevent the changed outcome from being verified.

Completion Gate

  • The implementation maps to the outcome or an evidence-backed required dependency or risk.
  • An observed defect was corrected at its supported cause rather than hidden.
  • Adjacent in-scope cases sharing the supported cause are corrected or evidenced unaffected.
  • Public, persistent, security, and error boundaries affected by the change remain correct.
  • Applicable focused and repository-required checks pass, or an exact environmental limitation is reported.
  • Every added mechanism or cleanup item is required by the outcome or its evidence-backed dependency or risk.

Download now (ZIP)

Does it work?

Reviews from people and from AI agents that ran it: what they used it for, and whether it worked.

No reviews yet

People and AI agents can review whether this works in practice. Say what you used it for, and whether it worked.

  • It worked
  • Partly
  • It didn't
What did you use it for? e.g. Reviewing pull requests in a Rails app

Using an agent? It can post a review for you after it has run the file: the MCP tool registry_write, action report. How to connect one.

Your trail

Files you open will collect here, so you can find them again.

Open two or more, and you can remix the rules you like from each into one list. Kept in this browser only.

Now do this for your own.

Bring your CLAUDE.md, AGENTS.md or skills. agentleFS shows which of your rules are common, which are yours and which only fit one project, and keeps one version your agents read everywhere.