Skip to content

refactor: deprecate steps/ folder pattern in job definitions - #367

Merged
nhorton merged 3 commits into
mainfrom
fix/remove-steps-folder-references
Apr 12, 2026
Merged

refactor: deprecate steps/ folder pattern in job definitions#367
nhorton merged 3 commits into
mainfrom
fix/remove-steps-folder-references

Conversation

@nhorton

@nhorton nhorton commented Apr 12, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Removes all references to the deprecated steps/ folder pattern across docs, AGENTS.md files, .deepreview rules, CI tests, and job instructions
  • Moves supplemental reference files from steps/ to job root directories
  • Updates the repair workflow to instruct agents to git rm step instruction files after inlining them into job.yml

Test plan

  • All 1289 tests pass
  • Verify CI e2e test correctly checks for inline instructions instead of step files
  • Run a repair workflow on a legacy job to confirm migration instructions work

🤖 Generated with Claude Code

nhorton and others added 3 commits April 12, 2026 17:09
Step instructions are now inlined in job.yml. This removes all references
to the steps/ folder pattern across documentation, AGENTS.md files,
.deepreview rules, CI tests, and job instructions. Supplemental reference
files are moved from steps/ to the job root directory.

The repair workflow retains steps/ references for migrating legacy jobs,
and now instructs agents to `git rm` step instruction files after inlining.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…review instructions

- Remove duplicate `**/conventions.md` in library/jobs/.deepreview include list
- Update engineer/.deepreview and template.deepreview to reference inline
  step instructions instead of deprecated step instruction files
- Add changelog entry for steps/ deprecation

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…ctions

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@nhorton
nhorton added this pull request to the merge queue Apr 12, 2026
Merged via the queue into main with commit 74614e9 Apr 12, 2026
5 checks passed
@nhorton
nhorton deleted the fix/remove-steps-folder-references branch April 12, 2026 23:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant