-
Notifications
You must be signed in to change notification settings - Fork 35
Fix repo-memory subdirectory support and metrics collector paths #7690
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
+255
−66
Conversation
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
Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
- Update push_repo_memory.cjs to recursively scan subdirectories - Enhance file-glob pattern matching to support ** wildcards - Fix metrics-collector.md to use correct /memory/default/ path - Fix workflow-health-manager.md to use correct /memory/default/ path - Recompile affected workflows Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
pelikhan
approved these changes
Dec 26, 2025
- Add tests for ** wildcard support in file-glob patterns - Test security implications of directory wildcards - Update metrics-collector to use metrics/** pattern - All JavaScript tests passing (2254 tests) Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
- Escape ** pattern in JSDoc comment to avoid syntax error - All JavaScript tests passing - Build successful Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Fix metrics collection infrastructure issues
Fix repo-memory subdirectory support and metrics collector paths
Dec 26, 2025
Contributor
|
@copilot merge main |
Contributor
|
@copilot merge main |
- Resolved conflicts in lock files - Kept changes to push_repo_memory.cjs in actions/setup/js/ - Recompiled all workflows successfully
Resolved merge conflicts: - Accepted deletion of pkg/workflow/js/*.cjs files (now synced from actions/setup/js) - Kept custom changes to push_repo_memory.cjs for subdirectory support - Recompiled all 124 workflows successfully - All lock files regenerated with updated build system Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Contributor
Author
Merged main branch successfully in commit 20d9f84. Resolved merge conflicts by:
All workflows compiled successfully. |
…ot/fix-metrics-collection-infrastructure
Contributor
Author
Merged latest main branch successfully in commit dd0498e. Integrated changes:
The branch is now fully up to date with main. |
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.
Fix Metrics Collection Infrastructure
Root Cause Analysis
Issue 1: Incorrect Directory Path in Workflow Instructions ✅
The
metrics-collector.mdworkflow instructions told the agent to write files to:But the correct path based on the memory ID "default" is:
Confusion: The
branch-name: memory/meta-orchestratorsis the git branch name, NOT the directory path. The directory path format is always/tmp/gh-aw/repo-memory-{ID}/memory/{ID}/.Issue 2: Push Script Doesn't Handle Subdirectories ✅
The
push_repo_memory.cjsscript only read top-level files and skipped directories, meaning files in subdirectories likemetrics/daily/*.jsonwere never pushed to the git branch.Fixes Applied
metrics/**for better compatibilityChanges Made
1. Enhanced push_repo_memory.cjs
**wildcard support for file-glob patterns2. Updated Workflow Instructions
/memory/meta-orchestrators/to/memory/default/metrics/**/*tometrics/**3. Comprehensive Testing
4. Main Branch Merges
Success Criteria
✅ Metrics Collector will run successfully with fixed paths
✅ Subdirectory files (metrics/daily/*.json) will be pushed to git branch
✅ File-glob patterns correctly filter files with ** support
✅ Workflow Health Manager can access metrics at correct location
✅ All tests passing
✅ Merged with main branch (twice - fully up to date)
Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.