Skip to content

fix(registry): bound expanded GitHub tree memory - #619

Merged
harlan-zw merged 1 commit into
mainfrom
chore/health-check-20261011
Oct 11, 2026
Merged

harlan-zw merged 1 commit into
mainfrom
chore/health-check-20261011

Conversation

@harlan-zw

Copy link
Copy Markdown
Collaborator

🔗 Linked issue

Related to #618

❓ Type of change

  • 📖 Documentation
  • 🐞 Bug fix
  • 👌 Enhancement
  • ✨ New feature
  • 🧹 Chore
  • ⚠️ Breaking change

📚 Description

The health check found repeated Worker memory failures while syncing aitofound/scienceide_env.
Tree expansion retained the discarded recursive response and could accumulate 250,000 entries.

Discard the partial response before expansion and cap retained paths and SHA strings at 8,388,608 characters.
Cap expanded entries at 100,000. If either limit is reached, return an empty truncated result to preserve existing Skills.

🤖 AI disclosure: Harlan Agent Kit modified this description. My AI open-source policy.

@harlan-github-agent harlan-github-agent Bot added harlan-agent-running An Agent holds a Task on this issue or pull request right now. harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. labels Oct 11, 2026
@harlan-zw harlan-zw added the harlan-agent-review Pull request triage requires an adversarial Review. Adding this label manually always forces one. label Oct 11, 2026
@harlan-github-agent

harlan-github-agent Bot commented Oct 11, 2026 •

Copy link
Copy Markdown
Contributor

🤖 MERGED

Harlan Agent Kit posted this automated review. It is not Harlan's personal review or approval. AI open source policy. Last updated: 2026-10-11 01:01 UTC.

GitHub merged this pull request.

  • Logged (10/100): The character budget undercounts total retained memory, but the 100,000-entry cap and conservative 8 MiB figure keep the real worst case within Worker limits. View code Optional follow-up: Optionally count a fixed per-entry overhead alongside the string budget for a tighter bound.
    • Ask an agent to verify and fix this finding

b2294df1-8328-4344-904d-ab624a7e2538

Selected findings run after merge and open separate pull requests.

@harlan-github-agent harlan-github-agent Bot removed the harlan-agent-review Pull request triage requires an adversarial Review. Adding this label manually always forces one. label Oct 11, 2026
@harlan-zw

Copy link
Copy Markdown
Collaborator Author

🤖 Harlan Agent Kit Agent checked live evidence that CI cannot collect:

  • Cloudflare recorded five memory terminations from October 10 at 23:01Z through October 11 at 00:21Z.
  • The last two timestamps match reserved scienceide_env jobs exactly. One row repeatedly records stale reservation recovery.
  • GitHub returned 62,574 entries with truncated: true for that Repository's recursive tree.
  • The protected health report and independent Cloudflare deployment metadata matched Worker 8367cf28-f2d8-4536-a480-cb4c1ac3a5c7.

The repair returns an empty truncated inventory when expansion exceeds its bounds. Existing Skills retain their source records.
Production memory recovery needs a fresh observation after deployment.

@harlan-github-agent harlan-github-agent Bot added harlan-agent-pending The automated Review is waiting on a gate for this head commit. and removed harlan-agent-running An Agent holds a Task on this issue or pull request right now. harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. labels Oct 11, 2026

@harlan-github-agent harlan-github-agent Bot 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.

Harlan Agent Kit posted this automated review. AI open source policy.

Automated Review: READY.

Read the Review results

@harlan-github-agent harlan-github-agent Bot added harlan-agent-ready The automated Review passed every gate on this head commit. and removed harlan-agent-pending The automated Review is waiting on a gate for this head commit. labels Oct 11, 2026
@harlan-zw
harlan-zw merged commit eb7b826 into main Oct 11, 2026
8 checks passed
@harlan-github-agent harlan-github-agent Bot removed the harlan-agent-ready The automated Review passed every gate on this head commit. label Oct 11, 2026
@harlan-zw
harlan-zw deleted the chore/health-check-20261011 branch October 11, 2026 01:23
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.

1 participant