agentleFS
Phelan164Phelan164/codex-howto/skills/engineering-loop/SKILL.md

engineering-loop

Skill15 starsChanged 3 months agoMITView on GitHub

Implement a repository code or configuration change through reproduction, focused checks, review, and evidence. Use for end-to-end features and fixes; not for explanation-only, review-only, research, content publishing, or production operations.

2 other projects use a version of this.

Adopted by

Most recently taken up by

The file

Show its rules and text

We split this skill into the rules it gives an agent

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

  • ≈0

    Common · estimate

    Many files like this have a rule of this kind.

  • ≈14

    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:

    Start with one agent. Delegate only when authorized and independent work or review coverage justifies the overhead.

    line 9
  2. Distinctive:

    Read applicable AGENTS.md files and repository documentation.

    line 15
  3. Distinctive:

    Before changing shared behavior, trace affected callers, consumers, and tests using targeted search or an available code graph.

    line 18
  4. Distinctive:

    Add a failing regression test first when practical.

    line 39
  5. Distinctive:

    If an unplanned path is necessary, record why and update the scope and checks.

    line 41
  6. Distinctive:

    Run affected consumer checks and repository-required broader checks.

    line 44
  7. Distinctive:

    If no test harness exists, use a small repeatable script or documented manual check with inputs, expected behavior, and observed results.

    line 53
  8. Distinctive:

    Keep a compact ledger of facts, changed files, commands, outcomes, and the next decision instead of full logs.

    line 58
  9. Distinctive:

    After two identical failures, stop blind retries and reclassify the cause.

    line 63
  10. Distinctive:

    If discovery or review repeats without new evidence, checkpoint the diff, unresolved issue, and next useful probe.

    line 65
  11. Distinctive:

    Stop and report a blocker when progress requires missing authority, secrets, unavailable infrastructure, an unauthorized destructive …

    line 67
  12. Distinctive:

    When an explicit loop budget is exhausted, stop with the latest judge evidence instead of silently expanding the budget.

    line 70
  13. Distinctive:

    Never make a failing check pass by weakening assertions, deleting coverage, hiding errors, or silently changing acceptance criteria.

    line 74
  14. Distinctive:

    Do not query production, deploy, migrate, merge, or publish unless the user explicitly authorizes that action.

    line 76

≈ “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

Engineering Loop

Deliver a verified change with minimal supervision and proportionate checking. Start with one agent. Delegate only when authorized and independent work or review coverage justifies the overhead. Follow applicable repository and team requirements; this skill adds no ticketing or mandatory-agent process.

Establish the baseline

  1. Read applicable AGENTS.md files and repository documentation.
  2. Record the requested behavior, constraints, and observable done conditions.
  3. Inspect the branch and working tree; preserve unrelated user-owned changes.
  4. Before changing shared behavior, trace affected callers, consumers, and tests using targeted search or an available code graph. Expand along relevant paths.
  5. Identify repository-native validation and run the smallest safe baseline that separates pre-existing failures from task regressions.

Resolve routine choices from evidence. Ask only for missing authority or a material decision: ambiguous acceptance, incompatible public behavior, or destructive/data-changing side effects not already authorized. Continue independent authorized work while awaiting an answer.

Load only the needed reference:

  • Long, cross-stack, or high-risk work: loop-contract.md.
  • Repeated iterations or explicit budgets: loop-policy.md.
  • Intermittent, performance, or unresolved defects: hard-debugging.md.
  • Requested or risk-justified separate review: independent-review.md.

Run the loop

  1. Reproduce the defect, or capture the current behavior for a feature.
  2. Choose the smallest coherent change and the evidence that will prove it.
  3. Add a failing regression test first when practical.
  4. Implement one bounded change and run the narrowest relevant check.
  5. If an unplanned path is necessary, record why and update the scope and checks.
  6. Classify failures as product, test, environment, assumption, or external blocker. Do not change product code to mask environment failures.
  7. Run affected consumer checks and repository-required broader checks.
  8. Review the complete diff first for task-contract compliance, then for code quality, regressions, security, and maintainability.
  9. Fix consequential findings and rerun checks affected by those fixes.

Once acceptance evidence and required checks pass, broaden or repeat checks only for new changes, failures, or unresolved concerns. Add tests when they prove behavior or a meaningful invariant, not merely mirror a low-impact edit.

If no test harness exists, use a small repeatable script or documented manual check with inputs, expected behavior, and observed results. Record missing automated coverage; a build alone does not prove runtime behavior. Do not add a large testing framework solely to satisfy the loop.

Keep a compact ledger of facts, changed files, commands, outcomes, and the next decision instead of full logs.

Stop conditions

  • After two identical failures, stop blind retries and reclassify the cause. Resume only with changed relevant state or an evidence-producing probe.
  • If discovery or review repeats without new evidence, checkpoint the diff, unresolved issue, and next useful probe. Do not restart the same cycle.
  • Stop and report a blocker when progress requires missing authority, secrets, unavailable infrastructure, an unauthorized destructive action, or an unresolved material product decision.
  • When an explicit loop budget is exhausted, stop with the latest judge evidence instead of silently expanding the budget. Without an explicit budget, continue only while a bounded next action can produce new evidence; productive iterations have no arbitrary fixed count.
  • Never make a failing check pass by weakening assertions, deleting coverage, hiding errors, or silently changing acceptance criteria.
  • Do not query production, deploy, migrate, merge, or publish unless the user explicitly authorizes that action.

Finish with evidence

Report changed behavior and files, regression or acceptance proof, exact check commands and outcomes, review disposition, and remaining limitations. Include the final reviewed revision when independent review was used. Never label blocked validation or unresolved consequential findings as complete. See evidence-example.md for a filled-in handoff.

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.