Repository navigation
Conversation
Co-Authored-By: Claude Code <noreply@anthropic.com>
github-actions
Bot
added
automation
documentation
pelikhan
marked this pull request as ready for review
October 1, 2026 23:54
pelikhan
deleted the
doc-healer/pi-quickstart-web-search-2026-10-01-aab5029bca9e70bb
branch
October 1, 2026 23:54
Contributor
Author
|
✅ Test Quality Sentinel completed test quality analysis. No test files were added or modified in this PR. Only documentation files changed (docs/src/content/docs/reference/tools.md, docs/src/content/docs/setup/quick-start.mdx). Test Quality Sentinel skipped.
|
Contributor
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
Both documentation updates accurately match current engine capabilities and validation behavior.
Review effort: Balanced
Findings: None
What changed in this PR
Updates engine documentation to clarify Pi authentication and web-search limitations.
Changes:
- Adds Pi account requirements to the quick start.
- Documents Gemini and Pi web-search behavior and MCP alternatives.
| File | Description |
|---|---|
docs/src/content/docs/setup/quick-start.mdx |
Clarifies Pi authentication prerequisites. |
docs/src/content/docs/reference/tools.md |
Documents Gemini and Pi web-search limitations. |
💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.
This was referenced Oct 2, 2026
Merged
Contributor
Author
|
🎉 This pull request is included in a new release. Release: |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Self-Healing Documentation Fixes
This PR was automatically created by the Daily Documentation Healer workflow.
It fixes two gaps that are still present in the tree. Both come from Deep Report issues that
github-actions[bot]auto-closed asnot_plannedwhen they expired. No merged PR fixed either one, and no closed-unmerged[docs]PR rejected either fix direction.Gaps Fixed
setup/quick-start.mdxnow mentions Pi. Pi has no account of its own: it uses Copilot, Anthropic, or OpenAI credentials, picked from the provider prefix inmodel:. This matches the Pi setup section in the same file.reference/tools.mdnow says that Gemini and Pi have no built-in web search. Declaringweb-search:with either engine has no effect, andgh aw compileprints a warning. The page links to Using Web Search for the MCP alternative.Source verification
pkg/workflow/gemini_engine.go:34andpkg/workflow/pi_engine.go:51both setWebSearch: false.pkg/workflow/agent_validation.go:277-281(validateWebSearchSupport) printsEngine '<id>' does not support the web-search toolas a compile warning when the capability is false.web-searchtool anywhere else.experimental: false), so the experimental-engine exemption does not apply.engines/gemini.md:53andengines/pi.md:97both say there is no nativetools.web-search, and the Pi auth rules come fromsetup/quick-start.mdx:175.Issues analyzed and skipped
29 closed
documentationissues in the 7-day window (since 2026-09-24):-l,--logical-repo/--force/--outputwording). Every flag is already documented incli.md, so there is no docs gap.docs/src/content/docs/: [deep-report] Add doc comments distinguishing JobInfo/JobData and ExperimentInfo/ExperimentData #64274 (Go doc comments), [deep-report] Fix 3 broken file-path references in .github/skills/*/SKILL.md #63483 (.github/skills/*/SKILL.md), [deep-report] Document transitive-closure test-loading patterns to prevent false-positive JS coverage audits #63231 (JS test-loading developer notes).[github-docs-seo]series ([github-docs-seo] Baseline Copilot eval recommends AW 0/10 times; add small Actions-doc cross-links #64408, [github-docs-seo] Improve GitHub Docs discoverability for Agentic Workflows in repository automation prompts #64171, [github-docs-seo] Baseline audit: AW recommended in 0 of 10 repo-automation prompts #63928, [github-docs-seo] Improve GitHub Docs routing so Copilot recommends GitHub Agentic Workflows for repository automation #63777, [github-docs-seo] Improve Agentic Workflows discoverability in automation docs from a 10-session baseline #63558, [github-docs-seo] Baseline Copilot eval: AW absent from 10/10 repository automation recommendations #63350, [github-docs-seo] Improve GitHub Agentic Workflows discoverability for repository automation prompts #63100, [github-docs-seo] Improve docs discoverability for GitHub Agentic Workflows in repository automation recommendations #62871 target docs.github.com),[spec-coverage][spec-coverage] Daily Spec Coverage Report — 36618565810 #64329, [spec-coverage] Daily Spec Coverage Report — 36343781363 #63872, [spec-coverage] Daily Spec Coverage Report — 35772872546 #62739,[spec-librarian][spec-librarian] Specification Audit — 2026-09-22 — 1 issue found #62659, older CLI-consistency reports ([cli-consistency] CLI Consistency Issues - 2026-09-25 #63447, [cli-consistency] CLI Consistency Issues - 2026-09-24 #63186, [cli-consistency] CLI Consistency Issues - 2026-09-22 #62660), and doc-healer [doc-healer] doc-healer DDUw improvement - integrity-filtered issue queries return empty #62496.The steering issue number for this run was empty, so there were no steering comments to read.
Root Cause
This is the third run in a row where the same DDUw gap caused the miss (see #64367 and #64596):
list_issuescall for opendocumentationissues during this run returned 3 items, and 7 more were[Filtered] ... removed by integrity policy. Deep Report issues are authored bygithub-actions[bot], so DDUw never sees them while they are open (the same pattern as [doc-healer] doc-healer DDUw improvement - integrity-filtered issue queries return empty #62496).gh-aw-expirestimer closes them asnot_planned, Step 1c says "Do not create documentation based solely on this issue". The cross-cutting fallback only covers "coverage or convention" gaps, so a single-file factual gap like these falls through.label:cookiefrom specific monitors. Deep Report issues do carrycookie, but Step 1d does not say it overrides the Step 1cnot_plannedrule, and its search hits the same integrity filter.💡 DDUw Improvement Suggestions
DDUw Improvement Suggestions
closed_byisgithub-actions[bot]and the body contains agh-aw-expiresmarker, treat the issue as unreviewed. Verify the gap against the current tree and fix it if it is still present."gh issue list. Tell DDUw to use those pre-fetched results as the source of truth when MCPlist_issues/search_issuesreports[Filtered] N item(s), instead of treating the filtered MCP result as complete.[deep-report]issues: they name an exact file and line and have a clear fix direction.--limit 20to 50. This window had 29 closeddocumentationissues, so a limit of 20 can drop the oldest ones..github/workflows/daily-doc-updater.md:210and:256still point to.github/instructions/documentation.instructions.md, which no longer exists. The guidelines live at.github/skills/documentation/SKILL.md.Related Issues
🤖 Generated with [Claude Code]((claude.com/redacted)