Migrate inline report-formatting guidance to the shared/reporting.md import - #52542
Merged
Conversation
…porting.md import Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Refactor inline report-formatting guidance to use shared/reporting.md
Migrate inline report-formatting guidance to the shared/reporting.md import
Aug 13, 2026
This was referenced Aug 13, 2026
pelikhan
marked this pull request as ready for review
August 13, 2026 19:14
Contributor
There was a problem hiding this comment.
Pull request overview
Centralizes report-formatting guidance by migrating 42 workflows to shared/reporting.md and regenerating compiled workflow metadata.
Changes:
- Added the shared reporting import and removed inline equivalents.
- Cleaned up formatting remnants and renumbered affected lists.
- Regenerated dependent lock files.
Show a summary per file
| File | Description |
|---|---|
.github/workflows/workflow-normalizer.lock.yml |
Regenerated metadata. |
.github/workflows/workflow-health-manager.lock.yml |
Regenerated metadata. |
.github/workflows/workflow-generator.lock.yml |
Regenerated metadata. |
.github/workflows/visual-regression-checker.lock.yml |
Regenerated metadata. |
.github/workflows/update-astro.lock.yml |
Regenerated metadata. |
.github/workflows/typist.md |
Migrates reporting guidance. |
.github/workflows/tidy.lock.yml |
Regenerated metadata. |
.github/workflows/terminal-stylist.md |
Migrates guidance and removes obsolete examples. |
.github/workflows/technical-doc-writer.lock.yml |
Regenerated metadata. |
.github/workflows/super-linter.lock.yml |
Regenerated metadata. |
.github/workflows/sub-issue-closer.lock.yml |
Regenerated metadata. |
.github/workflows/step-name-alignment.md |
Migrates reporting guidance. |
.github/workflows/stale-pr-cleanup.md |
Migrates reporting guidance. |
.github/workflows/squad-plan.lock.yml |
Regenerated metadata. |
.github/workflows/squad-game-planner.lock.yml |
Regenerated metadata. |
.github/workflows/spec-librarian.md |
Migrates guidance and renumbers guidelines. |
.github/workflows/sighthound-security-scan.lock.yml |
Regenerated metadata. |
.github/workflows/shared/reporting.md |
Documents the preferred import approach. |
.github/workflows/security-compliance.lock.yml |
Regenerated metadata. |
.github/workflows/scout.lock.yml |
Regenerated metadata. |
.github/workflows/schema-feature-coverage.lock.yml |
Regenerated metadata. |
.github/workflows/research.lock.yml |
Regenerated metadata. |
.github/workflows/repo-tree-map.lock.yml |
Regenerated metadata. |
.github/workflows/refiner.lock.yml |
Regenerated metadata. |
.github/workflows/pr-nitpick-reviewer.lock.yml |
Regenerated metadata. |
.github/workflows/pr-description-caveman.lock.yml |
Regenerated metadata. |
.github/workflows/poem-bot.lock.yml |
Regenerated metadata. |
.github/workflows/pdf-summary.lock.yml |
Regenerated metadata. |
.github/workflows/objective-impact-report.md |
Migrates reporting guidance. |
.github/workflows/notion-issue-summary.md |
Migrates reporting guidance. |
.github/workflows/necromancer.lock.yml |
Regenerated metadata. |
.github/workflows/metrics-collector.md |
Migrates reporting guidance. |
.github/workflows/issue-triage-agent.lock.yml |
Regenerated metadata. |
.github/workflows/issue-arborist.lock.yml |
Regenerated metadata. |
.github/workflows/gpclean.lock.yml |
Regenerated metadata. |
.github/workflows/go-pattern-detector.lock.yml |
Regenerated metadata. |
.github/workflows/functional-pragmatist.lock.yml |
Regenerated metadata. |
.github/workflows/firewall-escape.lock.yml |
Regenerated metadata. |
.github/workflows/example-failure-category-filter.lock.yml |
Regenerated metadata. |
.github/workflows/evoskill-evolver.lock.yml |
Regenerated metadata. |
.github/workflows/eslint-monster.lock.yml |
Regenerated metadata. |
.github/workflows/eslint-miner.lock.yml |
Regenerated metadata. |
.github/workflows/draft-pr-cleanup.md |
Migrates reporting guidance. |
.github/workflows/dictation-prompt.lock.yml |
Regenerated metadata. |
.github/workflows/dev.lock.yml |
Regenerated metadata. |
.github/workflows/detection-analysis-report.md |
Migrates reporting guidance. |
.github/workflows/dependabot-go-checker.lock.yml |
Regenerated metadata. |
.github/workflows/delight.md |
Migrates reporting guidance. |
.github/workflows/deepsec-security-scan.lock.yml |
Regenerated metadata. |
.github/workflows/daily-windows-terminal-integration-builder.md |
Migrates reporting guidance. |
.github/workflows/daily-token-consumption-report.md |
Migrates and deduplicates guidance. |
.github/workflows/daily-team-evolution-insights.md |
Adds the shared reporting import. |
.github/workflows/daily-syntax-error-quality.lock.yml |
Regenerated metadata. |
.github/workflows/daily-spec-coverage-kiro.lock.yml |
Regenerated metadata. |
.github/workflows/daily-security-red-team.md |
Migrates reporting guidance. |
.github/workflows/daily-secrets-analysis.md |
Migrates reporting guidance. |
.github/workflows/daily-secrets-analysis.lock.yml |
Regenerated compiled workflow. |
.github/workflows/daily-schema-audit-cursor.lock.yml |
Regenerated metadata. |
.github/workflows/daily-safe-outputs-conformance.md |
Migrates reporting guidance. |
.github/workflows/daily-safe-output-optimizer.md |
Migrates reporting guidance. |
.github/workflows/daily-repo-chronicle.md |
Migrates reporting guidance. |
.github/workflows/daily-reliability-review.md |
Migrates reporting guidance. |
.github/workflows/daily-regulatory.md |
Migrates reporting guidance. |
.github/workflows/daily-regression-audit-kiro.lock.yml |
Regenerated metadata. |
.github/workflows/daily-pr-review-cursor.lock.yml |
Regenerated metadata. |
.github/workflows/daily-performance-summary.md |
Migrates reporting guidance. |
.github/workflows/daily-observability-report.md |
Migrates reporting guidance. |
.github/workflows/daily-news.md |
Migrates reporting guidance. |
.github/workflows/daily-model-resolution.md |
Migrates reporting guidance. |
.github/workflows/daily-mcp-concurrency-analysis.md |
Migrates reporting guidance. |
.github/workflows/daily-max-ai-credits-test.md |
Migrates reporting guidance. |
.github/workflows/daily-go-test-parallelizer.lock.yml |
Regenerated metadata. |
.github/workflows/daily-fact.md |
Migrates reporting guidance. |
.github/workflows/daily-credit-limit-test.md |
Migrates reporting guidance. |
.github/workflows/daily-compiler-quality.md |
Migrates reporting guidance. |
.github/workflows/daily-cli-tools-tester.md |
Migrates reporting guidance. |
.github/workflows/daily-byok-ollama-test.md |
Migrates reporting guidance. |
.github/workflows/daily-assign-issue-to-user.md |
Migrates reporting guidance. |
.github/workflows/daily-architecture-diagram.md |
Migrates reporting guidance. |
.github/workflows/daily-agent-of-the-day-blog-writer.md |
Migrates reporting guidance. |
.github/workflows/craft.lock.yml |
Regenerated metadata. |
.github/workflows/copilot-session-insights.md |
Migrates reporting guidance. |
.github/workflows/copilot-pr-prompt-analysis.md |
Migrates reporting guidance. |
.github/workflows/copilot-pr-nlp-analysis.lock.yml |
Regenerated metadata. |
.github/workflows/copilot-pr-merged-report.lock.yml |
Regenerated metadata. |
.github/workflows/copilot-opt.lock.yml |
Regenerated metadata. |
.github/workflows/copilot-centralization-optimizer.lock.yml |
Regenerated metadata. |
.github/workflows/cli-version-checker.lock.yml |
Regenerated metadata. |
.github/workflows/chaos-pr-bundle-fuzzer.md |
Migrates reporting guidance. |
.github/workflows/bot-detection.md |
Migrates reporting guidance. |
.github/workflows/audit-workflows.md |
Migrates reporting guidance. |
.github/workflows/artifacts-summary.lock.yml |
Regenerated metadata. |
.github/workflows/approach-validator.lock.yml |
Regenerated metadata. |
.github/workflows/api-consumption-report.md |
Migrates reporting guidance. |
.github/workflows/ai-moderator.md |
Migrates reporting guidance. |
Review details
💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.
- Files reviewed: 172/172 changed files
- Comments generated: 2
- Review effort level: Balanced
| with: | ||
| title-prefix: "[daily-team-evolution] " | ||
| expires: 1d | ||
| - shared/reporting.md |
Comment on lines
+259
to
+261
| 4. **Use progressive disclosure**: Wrap details in `<details>` tags | ||
| 5. **One issue per run**: The `max: 1` limit ensures no issue spam | ||
| 6. **Skip if open**: The `skip-if-match` rule prevents duplicate issues |
github-actions Bot
added a commit
that referenced
this pull request
Aug 14, 2026
Extract a writeIfValid helper to remove repeated if(isValid...) writeEnvLine(...) patterns for trace-id/span-id/parent-span-id output and env propagation. Behavior unchanged; all log messages and file writes are identical. Source: recent activity around #52565 (shared prompt extraction) and #52542 (shared reporting import) motivated looking for similar duplication-reduction opportunities in actions/setup/js. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.
shared/reporting.mdalready single-sources report formatting rules (h3-only headers,<details>progressive disclosure, run-ID link format), but 42 workflows carried their own inline paraphrase instead of importing it — with drift already visible (>5 itemsvs>10 itemscollapse thresholds, differing<summary>conventions).Changes
shared/reporting.mdtoimports:(creating the block where absent, matching each file's existing list indentation) and removed the duplicated prose from the prompt body.terminal-stylist.md— orphaned<details>example and "Recommended Structure" list that only illustrated the removed rulesdaily-model-resolution.md/daily-token-consumption-report.md— residual sentence/bullet left behind after the heading was droppedspec-librarian.md— renumbered the guidelines list after removing item 4audit-workflows.md— restored a paragraph separatoragentic-token-optimizer.mdandci-doctor.mddeclaresource:and are read-only here; they need to change viagh aw update.shared/reporting.md: header comment now tells new-workflow authors to import rather than inline the guidance.make recompileregenerates all 285 lock files; the diff shows the sharedreportingskill block substituted for the removed prose with no other prompt content changes.