Skip to content

Comments

docs: Add check-fixer files to STRUCTURE.md [doc-updater]#891

Merged
jwbron merged 1 commit intomainfrom
egg/doc-update-check-fixer-structure
Feb 23, 2026
Merged

docs: Add check-fixer files to STRUCTURE.md [doc-updater]#891
jwbron merged 1 commit intomainfrom
egg/doc-update-check-fixer-structure

Conversation

@james-in-a-box
Copy link
Contributor

docs: Add check-fixer files to STRUCTURE.md [doc-updater]

Update STRUCTURE.md to reflect new per-check autofixer components from commit 650fca9.

Changes:

  • Added build-check-fixer-prompt.sh to action/ directory listing (marked deprecated build-autofixer-prompt.sh)
  • Added check-fixers.yml to shared/ directory listing (per-check config for non-LLM fixes, retries, model)
  • Added reusable-check-fixer.yml to workflows listing (marked deprecated reusable-autofix.yml)

The per-check autofixer introduces significant new capabilities:

  • Non-LLM mechanical fixes (ruff, shfmt, etc.) run before LLM fixer
  • Retry tracking per check with configurable max retries
  • Escalation to human when max retries exceeded
  • CI-driven loop model (fixer doesn't run checks locally)

The guide documentation (docs/guides/github-automation.md) was already comprehensively updated in the original commit. This PR updates only the structural documentation that enumerates project files.

Triggered by: 650fca9b

Authored-by: egg

Copy link
Contributor

@egg-reviewer egg-reviewer bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Documentation-only change to STRUCTURE.md — reviewed all 5 additions and 2 deletions.

Verification performed:

  • All three new files (shared/check-fixers.yml, action/build-check-fixer-prompt.sh, .github/workflows/reusable-check-fixer.yml) exist on main as of commit 650fca9.
  • Both deprecated files (build-autofixer-prompt.sh, reusable-autofix.yml) carry explicit # DEPRECATED headers in their source, consistent with the deprecation annotations added here.
  • Placement of check-fixers.yml in the shared/ tree listing is accurate — it sits between checkpoints/ and prompts/ in the actual filesystem.
  • Descriptions are factually correct per the file headers.

No issues found. Clean doc update.

— Authored by egg

@james-in-a-box
Copy link
Contributor Author

egg review completed. View run logs

@jwbron jwbron merged commit 8f9b119 into main Feb 23, 2026
22 checks passed
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