Daily Repo Goal Achiever: Add Daily File Diet workflow#194
Draft
github-actions[bot] wants to merge 1 commit intomainfrom
Draft
Daily Repo Goal Achiever: Add Daily File Diet workflow#194github-actions[bot] wants to merge 1 commit intomainfrom
github-actions[bot] wants to merge 1 commit intomainfrom
Conversation
Adds a language-agnostic workflow that monitors source file sizes and creates actionable refactoring issues when files exceed 500 lines. Adapted from Peli's Agent Factory (79% merge rate, 26 merged PRs). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
|
✅ Pull request created: #194 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR adds the Daily File Diet workflow from Phase 2 of the Daily Repo Goal Achiever task.
Phase 1: Consistency Check
No new consistency issues were found. The four docs pages previously missing the standard README backlink are already being addressed by open PR #192.
Phase 2: Daily File Diet Workflow
Source: daily-file-diet.md in gh-aw
Merge rate: 79% — 26 merged PRs out of 33 proposed in gh-aw, making it one of the more successful refactoring workflows in Peli's Agent Factory.
Why it's valuable: Large files are a universal code smell that slows every team down regardless of language. Files over 500 lines are harder to navigate, create more merge conflicts, mix concerns, and are harder to test in isolation. This workflow finds the largest non-test source file each weekday and creates a targeted refactoring issue with a concrete split strategy — one file at a time to keep the backlog manageable. If no file exceeds the threshold, it reports all-clear.
Generalization plan: The original workflow was deeply Go-specific: it used the Serena Go language server for semantic analysis, restricted scanning to the
pkg/directory, and used Go-specific bash commands. The adapted version:find,wc -l,grep) instead.go,.py,.ts,.js,.rb,.java,.rs,.cs,.cpp,.c) across the full repositorynode_modules,vendor,dist,build,target,__pycache__,venv,coverage)_test.go,.test.ts,.spec.js,test_*.py, etc.)Proposed name: Daily File Diet (same as original)
Files Added
workflows/daily-file-diet.md— workflow definitiondocs/daily-file-diet.md— documentation pageREADME.md— added entry in the "Code Improvement Workflows (by making changes)" section