Skip to content

ambient-context: remove noop-reminder imports, trim workflow prompts (~3,600 chars/run saved) - #43910

Merged
pelikhan merged 2 commits into
mainfrom
copilot/remove-shared-noop-reminder-imports
Jul 7, 2026
Merged

ambient-context: remove noop-reminder imports, trim workflow prompts (~3,600 chars/run saved)#43910
pelikhan merged 2 commits into
mainfrom
copilot/remove-shared-noop-reminder-imports

Conversation

Copilot AI commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

The runtime-injected system block already occupies 45–67% of every first request. Workflow-authored text was adding redundant content on top: shared/noop-reminder.md duplicated guidance already in the <safe-outputs> system block, daily-code-metrics.md carried a 1,238-char static chart table and a 899-char guidelines section that restated an already-imported shared file, linter-miner.md duplicated sub-agent output schemas that already live in the agent definitions, and shared/python-dataviz.md mixed env-setup commentary into the agent-visible body.

Changes

  • Remove shared/noop-reminder.md imports (172 workflows): drops {{#runtime-import shared/noop-reminder.md}} from 171 workflow bodies and the frontmatter imports: entry from daily-agent-of-the-day-blog-writer.md and daily-cache-strategy-analyzer.md

  • daily-code-metrics.md — chart specs → pre-agent step: the variant-conditional ### Required Charts table (~1,238 chars) is now written to /tmp/gh-aw/agent/chart-specs.md by a shell step keyed on GH_AW_EXPERIMENTS_OUTPUT_FORMAT; the agent body references it with one line

  • daily-code-metrics.md — remove ### Report Guidelines: 899-char section duplicated shared/reporting.md which is already imported transitively via shared/daily-audit-base.md

  • shared/python-dataviz.md — body trimmed to runtime contract: removes env-setup/library commentary (~800 chars), keeps dirs, data-separation rule, chart save pattern, and upload API (~500 chars total)

  • linter-miner.md — collapse Step 2 sub-agent schemas: replaces the verbose ### Discussion mining (826 chars) and ### Code pattern scanning (418 chars) sections in the main body with two bullet lines; full instructions already exist in the respective ## agent: definitions

…imming workflow prompts

- Remove {{#runtime-import shared/noop-reminder.md}} from 171 workflow bodies
- Remove shared/noop-reminder.md from frontmatter imports in
  daily-agent-of-the-day-blog-writer.md and daily-cache-strategy-analyzer.md
- Move Required Charts table to pre-agent-step in daily-code-metrics.md
- Remove redundant Report Guidelines section from daily-code-metrics.md
- Trim shared/python-dataviz.md body to runtime contract only (~500 chars saved)
- Relocate Discussion mining and Code pattern scanning schemas from
  linter-miner.md main body into agent definitions (~1,000 chars saved)
- Recompile all 258 workflows (0 errors)

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove shared/noop-reminder.md imports from workflows ambient-context: remove noop-reminder imports, trim workflow prompts (~3,600 chars/run saved) Jul 7, 2026
Copilot AI requested a review from pelikhan July 7, 2026 02:54
@pelikhan
pelikhan marked this pull request as ready for review July 7, 2026 06:08
Copilot AI review requested due to automatic review settings July 7, 2026 06:08
@pelikhan
pelikhan merged commit 1170fc4 into main Jul 7, 2026
@pelikhan
pelikhan deleted the copilot/remove-shared-noop-reminder-imports branch July 7, 2026 06:08

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.

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

🎉 This pull request is included in a new release.

Release: v0.82.3

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.

[ambient-context] Daily Ambient Context Optimizer - 2026-07-06

3 participants