Skip to content

fix(npm): cache failed package checks briefly - #611

Merged
harlan-zw merged 1 commit into
mainfrom
fix/npm-package-failure-cache
Oct 10, 2026
Merged

harlan-zw merged 1 commit into
mainfrom
fix/npm-package-failure-cache

Conversation

@harlan-zw

Copy link
Copy Markdown
Collaborator

🔗 Linked issue

❓ Type of change

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

📚 Description

Oversized archives and upstream failures repeated package checks because HTTP 503 bypassed the cache.
Cache unavailable results for 60 seconds before returning 503, then retry detection.
Keep found and absent results cached for one hour.

Package detection stores failed checks for 60 seconds before returning HTTP 503

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

@harlan-github-agent harlan-github-agent Bot added the harlan-agent-running An Agent holds a Task on this issue or pull request right now. label Oct 10, 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 10, 2026
@harlan-github-agent harlan-github-agent Bot added the harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. label Oct 10, 2026
@harlan-github-agent

harlan-github-agent Bot commented Oct 10, 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-10 09:15 UTC.

GitHub merged this pull request.

  • Logged (10/100): The pull request's behavioural screenshot is unavailable, so its claimed behavior has no inspectable visual evidence; the change itself is verified by code reading and the new unit test. View code Optional follow-up: Re-upload the attachment (or attach an accessible screenshot) to the pull request description before merge.
    • Ask an agent to verify and fix this finding
  • Logged (15/100): A corrupt or foreign edge-cache entry under the v3 key can throw inside windowsFor and 500 the package route until the entry expires, because the route omits the validate shape guard the cache module requires when windowsFor dereferences st View code Optional follow-up: Pass validate: value => typeof value === 'object' && value !== null && typeof (value as {_tag?: unknown})._tag === 'string' in the readThroughCache options.
    • Ask an agent to verify and fix this finding

f19bedc1-8d3c-451f-a305-56d4438c1d3a

Selected findings run after merge and open separate pull requests.

@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-review Pull request triage requires an adversarial Review. Adding this label manually always forces one. 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 10, 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-zw
harlan-zw merged commit 8a5600d into main Oct 10, 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 10, 2026
@harlan-zw
harlan-zw deleted the fix/npm-package-failure-cache branch October 10, 2026 09:57
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