Skip to content

fix(docs): reuse healthy skills index during Pages deploys - #45616

Merged
teknium1 merged 1 commit into
mainfrom
hermes/hermes-adb6f5f4
Jun 13, 2026
Merged

fix(docs): reuse healthy skills index during Pages deploys#45616
teknium1 merged 1 commit into
mainfrom
hermes/hermes-adb6f5f4

Conversation

@teknium1

Copy link
Copy Markdown
Contributor

Summary

Docs deploys now reuse a healthy skills index instead of recrawling every external skills source on each Pages build, preventing GitHub API quota exhaustion from blocking unrelated docs deploys.

Changes

  • deploy-site: add optional workflow inputs for a skills-index artifact run or forced rebuild.
  • deploy-site: download the scheduled Build Skills Index artifact first, then fall back to the currently live healthy index, and only crawl fresh if both are unavailable or explicitly forced.
  • skills-index: pass the successful scheduled run ID into deploy-site so scheduled refreshes deploy the exact artifact they just validated.

Validation

Check Result
Latest failed deploy build_skills_index hit GitHub API rate limit; github source collapsed to 0 and health check exited 2
Live index fallback downloaded and validated 92,232 skills locally
Artifact path downloaded run 27460361624 artifact and validated 92,192 skills locally
Workflow YAML parse deploy-site.yml and skills-index.yml parsed with PyYAML
Embedded shell syntax prepare-skills-index shell passed bash -n
Targeted tests scripts/run_tests.sh tests/scripts/test_build_skills_index_health.py → 2 passed
Whitespace git diff --check passed

Infographic

Healthy Index Reuse

@github-actions

Copy link
Copy Markdown
Contributor

🔎 Lint report: hermes/hermes-adb6f5f4 vs origin/main

ruff

Total: 0 on HEAD, 0 on base (➖ 0)

🆕 New issues: none

✅ Fixed issues: none

Unchanged: 0 pre-existing issues carried over.

ty (type checker)

Total: 10876 on HEAD, 10878 on base (✅ -2)

🆕 New issues (1):

Rule Count
invalid-assignment 1
First entries
tests/run_agent/test_credits_notices_toggle.py:76: [invalid-assignment] invalid-assignment: Object of type `None` is not assignable to attribute `_credits_session_start_micros` of type `int`

✅ Fixed issues (2):

Rule Count
unresolved-attribute 2
First entries
run_agent.py:2891: [unresolved-attribute] unresolved-attribute: Object of type `Self@get_credits_spent_micros` has no attribute `_credits_session_start_micros`
tests/run_agent/test_credits_notices_toggle.py:76: [unresolved-attribute] unresolved-attribute: Unresolved attribute `_credits_session_start_micros` on type `AIAgent`

Unchanged: 5707 pre-existing issues carried over.

Diagnostics are surfaced as warnings — this check never fails the build.

@alt-glitch alt-glitch added type/bug Something isn't working P2 Medium — degraded but workaround exists labels Jun 13, 2026
@teknium1
teknium1 merged commit 4373e80 into main Jun 13, 2026
25 checks passed
@teknium1
teknium1 deleted the hermes/hermes-adb6f5f4 branch June 13, 2026 13:46
AIalliAI pushed a commit to AIalliAI/Hermes that referenced this pull request Jun 14, 2026
T02200059 pushed a commit to T02200059/hermes-agent that referenced this pull request Jun 18, 2026
waefrebeorn pushed a commit to waefrebeorn/slermes that referenced this pull request Jul 2, 2026
habarmc1223-sudo pushed a commit to habarmc1223-sudo/hermes-agent-fluxmem that referenced this pull request Jul 8, 2026
santhreal pushed a commit to santhreal/hermes-agent that referenced this pull request Jul 13, 2026
donbowman pushed a commit to donbowman/hermes-agent that referenced this pull request Jul 13, 2026
Gravezzz pushed a commit to Gravezzz/hermes-agent that referenced this pull request Jul 21, 2026
leewenjie pushed a commit to leewenjie/hermes-agent that referenced this pull request Aug 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

P2 Medium — degraded but workaround exists type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants