Skip to content

Conversation

@ianhe8x
Copy link
Contributor

@ianhe8x ianhe8x commented Dec 15, 2025

Summary by CodeRabbit

  • Documentation
    • Added new AI documentation entries to the sidebar.
    • GraphQL Agent documentation is now available.
    • Hermes feature documentation marked as coming soon.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Dec 15, 2025

Walkthrough

Updated the AI section sidebar configuration to activate a GraphQL Agent documentation link and add a placeholder entry for Hermes (marked Coming Soon). Created a new placeholder documentation file for Hermes with a "Coming Soon" notice.

Changes

Cohort / File(s) Summary
Sidebar Configuration
docs/.vuepress/sidebar.ts
Replaced commented-out "SubQuery GraphQL Agent" entry with two active sidebar items: GraphQL Agent (linking to ai/graphql_agent.md) and Hermes (Coming) (linking to ai/hermes.md).
Documentation
docs/ai/hermes.md
Added new placeholder documentation file with "## Coming Soon" header.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • Update GraphQL Agent documentation #668: Surfaces GraphQL Agent documentation in the AI sidebar and updates related documentation references, aligning with this PR's activation of GraphQL Agent links and sidebar restructuring.

Poem

🐰 The docs take shape with paths so clear,
GraphQL Agent now appears,
Hermes waits just down the road,
"Coming Soon" adorns the code,
The sidebar hops to guide the way,
Documentation blooms today! 📚✨

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'update hermes subnet' is vague and does not clearly describe the actual changes made in the pull request, which involve adding documentation for a Hermes feature and updating the sidebar navigation. Use a more descriptive title such as 'Add Hermes documentation and update sidebar navigation' to better convey the specific changes being made.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch hermes-subnet

Comment @coderabbitai help to get the list of available commands and usage tips.

@ianhe8x ianhe8x merged commit bd8df2c into master Dec 15, 2025
2 of 3 checks passed
@ianhe8x ianhe8x deleted the hermes-subnet branch December 15, 2025 05:14
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 0

🧹 Nitpick comments (1)
docs/ai/hermes.md (1)

1-1: Consider slightly richer placeholder copy

The placeholder is valid, but you might add a brief one‑liner (e.g., what Hermes is or that docs will arrive with the subnet launch) so users hitting this page get a tiny bit more context.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 14bba7b and bd08e86.

📒 Files selected for processing (2)
  • docs/.vuepress/sidebar.ts (1 hunks)
  • docs/ai/hermes.md (1 hunks)
🔇 Additional comments (1)
docs/.vuepress/sidebar.ts (1)

748-755: Sidebar entries look structurally correct; verify localized docs exist for all supported locales

The new /ai/ entries are consistent with the rest of the config and the links look syntactically correct.

However, both graphql_agent.md and hermes.md currently exist only in the default locale. The sidebar config uses ${locale}/ai/graphql_agent.md and ${locale}/ai/hermes.md, which means non-default locales (e.g., zh, es) will attempt to load /zh/ai/hermes.md and /zh/ai/graphql_agent.md—files that don't exist yet. This will result in 404s for users viewing these sections in non-default languages.

If localized docs are planned, add the necessary files or stubs. If this is intentional (e.g., these features are only documented in English for now), either hard-code the default locale path for these links or document the limitation.

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.

2 participants