fix(creator): raise max_history_messages + repair refresh-cache CI (#91)

* fix(creator): raise max_history_messages to 80 for polling workflows

Creator Hand's async video_generate path polls video_status every 15-20s
until completion (1-3 min typical), consuming ~5-15 turns per video
request. Combined workflows (video + TTS + music) plus normal back-and-
forth cross the kernel default of 40 messages quickly, which surfaced
in user logs as:

  WARN run_agent_loop: Trimming old messages at safe turn boundary
    agent=creator:creator-hand total_messages=41 trimming=2
  INFO run_agent_loop: prompt cache metrics for turn
    hit_ratio=0.0 creation=0 read=0

Every turn was hitting the trim cap and invalidating the prompt-cache
prefix. 80 covers ~30 polling iterations plus a comfortable pre-context
window without runaway memory growth. Other hands keep the default 40.

* ci(refresh-cache): open PR instead of pushing directly to main

Branch protection on `main` started rejecting the workflow's auto-commit
with GH006 "Changes must be made through a pull request" — see run
25632824585 on 2026-05-10 against commit 6785807 (the first push that
hit the tightened protection). Direct push is precisely what the file's
own security comment (#1) warns against ("Compromised maintainer pushes
a malicious plugins-index.json directly to main. Mitigation: GitHub
branch protection on main requires PR review"), so the fix preserves
that gate rather than working around it.

The workflow now creates a short-lived `automation/refresh-indexes-<sha>`
branch, commits the regen there, pushes, and opens a PR back to main
via `gh pr create`. Maintainers see a one-click squash-merge.

Permissions: add `pull-requests: write` to the existing `contents: write`
so `gh pr create` can be authorised through the default GITHUB_TOKEN.

The post-merge run on the index PR is a no-op (no diff under
`hands/**`, `plugins/**`, etc. between consecutive states), so no
`[skip ci]` marker is needed and no loop is possible.

Without this fix, every content PR landing on main leaves
plugins-index.json + registry-index.json stale, blocking new agents and
hands from reaching daemons until a maintainer manually regenerates.

* fix(hands): raise max_history_messages on long-workflow coordinators

Three hand coordinators have workflows that routinely exceed the kernel
default history cap on a single user turn:

- researcher (max_iterations=80) — deep web_search → web_fetch →
  summarize loops with multi-source synthesis. 80 iterations × ~4
  messages each → 200+ messages per user turn. Set to 120.
- devops    (max_iterations=60) — incident response and CI/CD fan out
  into long shell_exec chains (logs, retries, post-mortems). Set to 80.
- predictor (max_iterations=60) — long reasoning chains accumulating
  signals across many web/knowledge queries, with scheduled re-checks
  referring back. Set to 80.

Creator's existing override is rephrased "raise above the kernel
default" so the comment stays correct regardless of the order this PR
and the upstream kernel-default bump (librefang side) land in.

Other hands (lead/linkedin/reddit/clip/analytics/apitester/browser/
collector/strategist) stay on the kernel default; the upstream bump
covers them.
This commit is contained in:
Evan authored and GitHub committed 2026-05-12 08:55:22 +09:00
1 parent 6785807633
commit 651ff1b34d
5 files changed
+61 -5

No files matched your search

+36 -5
View File
@@ -56,7 +56,8 @@ on:
workflow_dispatch: workflow_dispatch:
permissions: permissions:
contents: write # commit regenerated index files back contents: write # push the regen to a side branch
pull-requests: write # open the regen PR for maintainer merge
jobs: jobs:
refresh: refresh:
@@ -120,17 +121,47 @@ jobs:
console.log("Signature verifies OK against committed pubkey."); console.log("Signature verifies OK against committed pubkey.");
' '
- name: Commit regenerated indexes if changed # Direct `git push` to `main` is rejected by branch protection
# (GH006: "Changes must be made through a pull request"), which is
# the intended security model documented at the top of this file
# (mitigation #1). Open a PR with the regen instead so the same
# human-review gate applies to bot-authored index updates. The PR
# body links back to the triggering content commit so reviewers can
# eyeball the regen against the source change.
- name: Open PR with regenerated indexes if changed
env:
GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
run: | run: |
git config user.name "github-actions[bot]" git config user.name "github-actions[bot]"
git config user.email "github-actions[bot]@users.noreply.github.com" git config user.email "github-actions[bot]@users.noreply.github.com"
git add plugins-index.json plugins-index.json.sig registry-index.json git add plugins-index.json plugins-index.json.sig registry-index.json
if git diff --cached --quiet; then if git diff --cached --quiet; then
echo "indexes already up-to-date" echo "indexes already up-to-date"
else exit 0
git commit -m "chore: regenerate registry indexes"
git push
fi fi
branch="automation/refresh-indexes-${GITHUB_SHA::8}"
git switch -c "$branch"
git commit -m "chore: regenerate registry indexes for ${GITHUB_SHA::8}"
git push -u origin "$branch"
gh pr create \
--base main \
--head "$branch" \
--title "chore: regenerate registry indexes for ${GITHUB_SHA::8}" \
--body "Auto-generated by \`.github/workflows/refresh-cache.yml\` after $GITHUB_SHA.
Regenerates the two indexes the registry-worker ingests:
- \`plugins-index.json\` + \`plugins-index.json.sig\` (signed)
- \`registry-index.json\`
The signature was verified against the committed pubkey in the
generating run before this PR was opened (see the run linked
on the commit).
Branch protection on \`main\` blocks direct push (the documented
security model — see the header of \`refresh-cache.yml\`), so
this PR carries the regen for maintainer review and merge.
Squash-merge is safe; \`[skip ci]\` is not needed since the
regen run on the merge commit will be a no-op."
- name: Trigger worker refresh - name: Trigger worker refresh
env: env:
+8
View File
@@ -175,6 +175,14 @@ model = "default"
max_tokens = 8192 max_tokens = 8192
temperature = 0.5 temperature = 0.5
max_iterations = 30 max_iterations = 30
# Raise the history cap above the kernel default. Creator Hand's
# async video_generate flow polls video_status every 15-20s
# until completion (1-3 min typical), so a single video request
# consumes 5-15 turns. Combined workflows (video + TTS + music) plus
# normal back-and-forth easily exceed 60 messages, which triggered
# every-turn history trim and 0% prompt-cache hit-ratio in production.
# 80 covers ~30 polls plus a comfortable pre-context window.
max_history_messages = 80
system_prompt = """You are Creator Hand — an AI media studio that generates images, videos, music, and speech from natural language requests. system_prompt = """You are Creator Hand — an AI media studio that generates images, videos, music, and speech from natural language requests.
## Available Tools ## Available Tools
+5
View File
@@ -274,6 +274,11 @@ model = "default"
max_tokens = 16384 max_tokens = 16384
temperature = 0.2 temperature = 0.2
max_iterations = 60 max_iterations = 60
# Raise the history cap above the kernel default. Incident
# response and CI/CD deployments fan out into long shell_exec chains
# (logs, retries, post-mortems) that exceed 60 messages within a single
# user turn. 80 buys headroom without doubling the cost.
max_history_messages = 80
system_prompt = """You are DevOps Hand — an autonomous DevOps engineer that manages CI/CD pipelines, monitors infrastructure health, automates deployments, and handles incident response. system_prompt = """You are DevOps Hand — an autonomous DevOps engineer that manages CI/CD pipelines, monitors infrastructure health, automates deployments, and handles incident response.
## Phase 0 — Environment Detection (ALWAYS DO THIS FIRST) ## Phase 0 — Environment Detection (ALWAYS DO THIS FIRST)
+6
View File
@@ -212,6 +212,12 @@ model = "default"
max_tokens = 16384 max_tokens = 16384
temperature = 0.5 temperature = 0.5
max_iterations = 60 max_iterations = 60
# Raise the history cap above the kernel default. Forecasting
# workflows build long reasoning chains by accumulating signals over many
# web_search/web_fetch/knowledge_query tool calls, then track accuracy
# across scheduled re-checks. 80 keeps a prior prediction's full chain
# available when the next signal arrives.
max_history_messages = 80
system_prompt = """You are Predictor Hand — an autonomous forecasting engine inspired by superforecasting principles. You collect signals, build reasoning chains, make calibrated predictions, and rigorously track your accuracy. system_prompt = """You are Predictor Hand — an autonomous forecasting engine inspired by superforecasting principles. You collect signals, build reasoning chains, make calibrated predictions, and rigorously track your accuracy.
## Phase 0 — Platform Detection & State Recovery (ALWAYS DO THIS FIRST) ## Phase 0 — Platform Detection & State Recovery (ALWAYS DO THIS FIRST)
+6
View File
@@ -203,6 +203,12 @@ model = "default"
max_tokens = 16384 max_tokens = 16384
temperature = 0.3 temperature = 0.3
max_iterations = 80 max_iterations = 80
# Raise the history cap above the kernel default. Deep research workflows do
# extensive web_search → web_fetch → summarize
# loops with multi-source synthesis: 80 iterations × ~4 messages each
# easily produces 200+ messages per user turn. 120 keeps ~1.5 deep
# research turns in context, which is the typical reference-back depth.
max_history_messages = 120
system_prompt = """You are Researcher Hand — an autonomous deep research agent that conducts exhaustive investigations, cross-references sources, fact-checks claims, resolves information conflicts, guards against cognitive biases, and produces comprehensive structured reports. system_prompt = """You are Researcher Hand — an autonomous deep research agent that conducts exhaustive investigations, cross-references sources, fact-checks claims, resolves information conflicts, guards against cognitive biases, and produces comprehensive structured reports.
## Phase 0 — Platform Detection & Context (ALWAYS DO THIS FIRST) ## Phase 0 — Platform Detection & Context (ALWAYS DO THIS FIRST)