Skip to content

reduce instructions-janitor max lines target from 500 to 400#41673

Merged
pelikhan merged 2 commits into
mainfrom
copilot/reduce-instructions-janitor-max-lines
Jun 26, 2026
Merged

reduce instructions-janitor max lines target from 500 to 400#41673
pelikhan merged 2 commits into
mainfrom
copilot/reduce-instructions-janitor-max-lines

Conversation

Copilot AI commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Lowers all per-file line limits in the Instructions Janitor workflow from 500 to 400 to keep instruction files more compact for agentic consumption.

Changes

  • .github/workflows/instructions-janitor.md — updated File Layout Goals table and size-audit enforcement rule:
    • syntax-*.md, safe-outputs-*.md, create-agentic-workflow.md, and any new .github/aw/*.md file: < 500< 400 lines
    • Size audit: "any file over 500 lines must be split" → 400
    • Workflow description updated to reflect new threshold
  • .github/workflows/instructions-janitor.lock.yml — recompiled from updated source

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Reduce instructions-janitor max lines target from 500 to 400 reduce instructions-janitor max lines target from 500 to 400 Jun 26, 2026
Copilot AI requested a review from pelikhan June 26, 2026 12:07
@pelikhan pelikhan marked this pull request as ready for review June 26, 2026 12:11
Copilot AI review requested due to automatic review settings June 26, 2026 12:11
@pelikhan pelikhan merged commit 727f3b8 into main Jun 26, 2026
@pelikhan pelikhan deleted the copilot/reduce-instructions-janitor-max-lines branch June 26, 2026 12:11

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This pull request updates the Instructions Janitor agent workflow guidance to tighten the target maximum length for instruction files, reducing the per-file line limit threshold from 500 to 400 to keep .github/aw/*.md content more compact for agentic loading.

Changes:

  • Updated the workflow description and the “File Layout Goals” table thresholds from < 500 to < 400 where applicable.
  • Updated the size-audit rule to require splitting/rewriting any file over 400 lines.
  • Recompiled the generated workflow lock file to reflect the updated source markdown.
Show a summary per file
File Description
.github/workflows/instructions-janitor.md Lowers stated per-file size targets and the size-audit enforcement rule from 500 to 400 lines.
.github/workflows/instructions-janitor.lock.yml Regenerated compiled workflow output to incorporate the updated markdown workflow content/metadata.

Review details

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Low

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.

3 participants