Skip to content

docs: salvage 28 validated docs PRs from the older backlog (cluster 2) - #49944

Merged
teknium1 merged 31 commits into
mainfrom
hermes/hermes-7b9b9601
Jun 21, 2026
Merged

docs: salvage 28 validated docs PRs from the older backlog (cluster 2)#49944
teknium1 merged 31 commits into
mainfrom
hermes/hermes-7b9b9601

Conversation

@teknium1

Copy link
Copy Markdown
Contributor

Summary

Second docs-backlog cluster — salvages 28 validated documentation PRs onto current main, each contributor commit cherry-picked with authorship preserved (rebase-merge). All premises re-verified against current main by parallel validation before inclusion. (Follows the first cluster #49914.)

Why this cluster exists

The initial docs sweep capped at the 1000 most-recent open PRs and missed roughly half the backlog (there are ~2000 open PRs). This cluster covers the older docs PRs (mostly Jun 14–17) that fell below that cap.

Salvaged PRs (28)

Prose / guides / config:
#46846 @GauravPatil2515 · #46873 @Sworntech-dev · #47046 @x7peeps · #47097 @m24927605 · #47296 @bbopen · #47379 @DrZM007 · #47501 @Sworntech-dev · #47637 @x7peeps · #49923 @asimons81 · #46254 @skyc1e · #46352 @BBCrypto-web · #46383 @lkz-de · #46499 @andressommerhoff · #46777 @gdeyoung · #46933 @jvradahellys24-art · #47850 @HwangJohn · #48409 @aieng-abdullah · #48449 @whoislikemiha · #48758 @virtuadex · #49057 @anderskev · #49216 @teknium1 · #49277 @Tortugasaur · #49392 @alloevil · #49675 @loes5050 · #49779 @BBCrypto-web · #49802 @Sworntech-dev · #49814 @namredips · #49877 @BBCrypto-web

Each was confirmed to fix a real gap on current main (broken link, missing placeholder, stale value, undocumented hook/feature, etc.). A handful carry small validated code changes alongside docs (search_files newline warning #46499, spotify re-auth quarantine #48409, hook docstrings) — each self-contained and validated.

Notes

Validation

  • Docs-dominant cluster; per-PR premise verified against current origin/main.
  • Stale-base gate: 0 commits behind origin/main.
  • Authorship audit: no tool identities; every commit credited to its contributor.

skyc1e and others added 30 commits June 20, 2026 20:59
Clarify that session_search is secondary context and direct source identifiers must be inspected first when accessible. Add regression coverage for the tool description.
…scovery rules

Adds a new 'Project Context Files' section to the hermes-agent skill
explaining the priority order and discovery rules for .hermes.md,
AGENTS.md, CLAUDE.md, and .cursorrules. Specifically clarifies:

- .hermes.md walks parents up to the git root (good for monorepos)
- AGENTS.md / agents.md is cwd-only (portable to other agents)
- The 20K cap and head+tail truncation strategy
- The threat-pattern scanner behavior (blocks content, not file)
- What --ignore-rules actually skips (everything)

Also fixes an inaccurate docstring in agent/agent_init.py for
skip_context_files — the previous text only mentioned SOUL.md,
AGENTS.md, and .cursorrules, but the actual behavior (per
build_context_files_prompt and the --ignore-rules CLI flag) skips
all of them plus .hermes.md and CLAUDE.md.

Refs: #46775
- Complete README.es.md (full Spanish translation of README)
- Add CONTRIBUTING.es.md (Spanish contributing guide)
- Add SECURITY.es.md (Spanish security policy)
- Fix remaining English strings in locales/es.yaml (resume Matrix section)
- Add Spanish badge to README.md

All 47 i18n tests pass, including catalog key parity and placeholder parity.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
The SKILL.md template in CONTRIBUTING.md was missing the Prerequisites
and How to Run sections, even though the "modern section order"
guidance immediately below it lists both as required.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: HwangJohn <angelic805@gmail.com>

Co-authored-by: OpenAI Codex <codex@openai.com>
resolve_spotify_runtime_credentials() called _refresh_spotify_oauth_state()
without a try/except, so a terminal failure (HTTP 400/401, invalid_grant,
refresh_token_reused) raised AuthError but left the dead refresh_token in
auth.json. Every subsequent session re-read and retried the same token over
the network, failing identically each time.

Fix: wrap the refresh call and, when exc.relogin_required is True and a
refresh_token is present, clear the dead OAuth fields (access_token,
refresh_token, expires_at, expires_in, obtained_at) and write a
last_auth_error quarantine marker to auth.json before re-raising. The next
call sees no access_token and fails fast with spotify_access_token_missing —
no network retry — and the user is prompted to re-authenticate.

Mirrors the quarantine pattern already in place for Nous, xAI-OAuth,
Codex-OAuth (#28116, #28118), and MiniMax-OAuth (#28119).
…valid_grant test

- Remove the 'you only log in once per machine' claim from spotify.md
  and document the ~6-month refresh token expiry with re-auth instructions
- Add test_client_wraps_invalid_grant_as_spotify_auth_required_error to
  confirm SpotifyClient wraps AuthError(code=spotify_refresh_invalid_grant)
  into SpotifyAuthRequiredError with a user-facing message

Refs: #28155
The terminal backend onboarding step pointed at
/docs/developer-guide/environments, which no longer exists. Point it at
the live docs page /docs/user-guide/configuration#terminal-backend-configuration.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…docstring

The MCP discovery wait is now bounded by the config-driven mcp_discovery_timeout
(default 1.5s), not the old 0.75s flat value. Updates the _schedule_mcp_late_refresh
docstring that still cited ~0.75s after #49208 made the bound configurable.
The shell-hook stdin payload's extra object contains event-specific
kwargs, but the docstring only mentioned the field without listing
what each event actually puts inside it.

Add a reference table covering post_tool_call, pre_tool_call,
on_session_start, on_session_end, and subagent_stop — the five
hook sites that emit extra keys beyond the top-level payload.

Closes #49370
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Brings the antigravity-cli skill to parity with the codex / claude-code
delegation playbooks. Additive only — auth/sandbox/plugin/settings content
is unchanged.

- New 'Delegation patterns' section: one-shot, background bounded runs,
  interactive PTY+tmux, parallel worktree fan-out, and an orchestration
  boundary note (agy is a worker backend / reviewer, not a coordination
  primitive).
- Documents the two ways agy -p differs from claude-code: plain-text
  output (no --output-format json / result envelope) and bounding via
  --print-timeout rather than a nonexistent --max-turns. Mirrored into
  Pitfalls.
- Bumps version 0.1.0 -> 0.2.0.
The MCP section pointed to docs/mcp.md, which does not exist. Point it
to website/docs/user-guide/features/mcp.md, matching the existing
hooks.md reference convention in the same file.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This was referenced Jun 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

P3 Low — cosmetic, nice to have type/docs Documentation improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.