-
Notifications
You must be signed in to change notification settings - Fork 132
[docs] Unbloat common-issues.md troubleshooting documentation #14994
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Reduced common-issues.md from 419 to 345 lines (18% reduction) by: - Consolidating redundant explanations - Streamlining troubleshooting steps - Removing verbose setup instructions - Converting excessive prose to concise formats All essential information preserved while improving readability. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Reduces verbosity in common-issues.md to make troubleshooting guidance more direct and scannable while retaining key links and examples.
Changes:
- Condensed installation and Codespaces troubleshooting into shorter prose.
- Streamlined organization/repository policy guidance and verification steps.
- Tightened debugging, network configuration, and troubleshooting sections.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| > [!NOTE] | ||
| > The install script works in restricted network environments (like GitHub Copilot agent runtime or corporate networks with MITM proxies) by using GitHub's "latest" release redirect, which doesn't require API access. Simply run the script without specifying a version to install the latest release. | ||
| The installer works in restricted networks (Codespaces, MITM proxies) and installs to `~/.local/share/gh/extensions/gh-aw/gh-aw`. |
Copilot
AI
Feb 11, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This sentence conflates Codespaces with restricted network environments; earlier the doc describes Codespaces issues as permission/auth limitations rather than network restrictions. Consider removing “Codespaces” from the restricted-network parenthetical (or rewording to distinguish permissions vs network constraints) to avoid misleading guidance.
| The installer works in restricted networks (Codespaces, MITM proxies) and installs to `~/.local/share/gh/extensions/gh-aw/gh-aw`. | |
| The installer works in restricted network environments (for example, MITM proxies) and installs to `~/.local/share/gh/extensions/gh-aw/gh-aw`. |
| ### Import File Not Found | ||
|
|
||
| Import paths are relative to repository root. Verify the file exists and is committed (`git status`). Example paths: `.github/workflows/shared/tools.md` (from repo root) or `shared/security-notice.md` (relative to `.github/workflows/`). | ||
| Import paths are relative to repository root. Verify the file exists with `git status`. Examples: `.github/workflows/shared/tools.md` or `shared/security-notice.md`. |
Copilot
AI
Feb 11, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
git status doesn’t verify that a file exists at a given path; it only reports tracked/untracked/modified state. For existence/path validation, use something like ls <path>, test -f <path>, or git ls-files --error-unmatch <path> (if you specifically want “exists and tracked”).
| Import paths are relative to repository root. Verify the file exists with `git status`. Examples: `.github/workflows/shared/tools.md` or `shared/security-notice.md`. | |
| Import paths are relative to repository root. Verify the file exists with `ls <path>` (or, if it must be tracked, `git ls-files --error-unmatch <path>`). Examples: `.github/workflows/shared/tools.md` or `shared/security-notice.md`. |
| **Problem**: GitHub Projects reserves field types like `REPOSITORY` that cannot be updated via API. | ||
|
|
||
| **Solution**: Use different field names to avoid conflicts with GitHub's built-in types: | ||
| **Solution**: Use alternative field names (`repo`, `source_repository`, `linked_repo`) instead of `repository`: |
Copilot
AI
Feb 11, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The prose lists multiple alternative field name examples (repo, source_repository, linked_repo) but the code sample now only demonstrates repo. Either adjust the text to mention just repo (or “e.g., repo”) or expand the example to include the other suggested alternatives to keep the guidance consistent.
| **Solution**: Use alternative field names (`repo`, `source_repository`, `linked_repo`) instead of `repository`: | |
| **Solution**: Use an alternative field name (for example, `repo`) instead of `repository`: |
| command: "node" | ||
| args: ["./server.js"] | ||
| ``` | ||
| Use local MCP servers if HTTP connections timeout (`command: "node"`, `args: ["./server.js"]`). |
Copilot
AI
Feb 11, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Embedding YAML keys inline is harder to scan and may not be copy/pasteable as-is. Consider restoring a minimal fenced YAML example (even a very short one) so readers can directly copy a working mcp-servers: configuration.
| ### Cache Not Restoring | ||
|
|
||
| Verify cache key patterns match and note that caches expire after 7 days: | ||
| Verify cache key patterns match. Caches expire after 7 days. |
Copilot
AI
Feb 11, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
GitHub Actions cache eviction is typically based on inactivity (e.g., not accessed for a period) rather than a simple fixed “expires after 7 days” statement. Reword to something like “Caches may be evicted after ~7 days of inactivity” (or match the exact wording from GitHub’s docs) to avoid implying a hard TTL.
| Verify cache key patterns match. Caches expire after 7 days. | |
| Verify cache key patterns match. Caches may be evicted after about 7 days of inactivity. |
| ### Debugging Strategies | ||
|
|
||
| Enable verbose compilation (`gh aw compile --verbose`), set `ACTIONS_STEP_DEBUG = true` for debug logging, inspect generated lock files (`cat .github/workflows/my-workflow.lock.yml`), check MCP configuration (`gh aw mcp inspect my-workflow`), and review logs (`gh aw logs my-workflow` or `gh aw audit RUN_ID`). | ||
| Enable verbose compilation (`--verbose`), set `ACTIONS_STEP_DEBUG = true`, inspect lock files, check MCP config (`gh aw mcp inspect`), and review logs. |
Copilot
AI
Feb 11, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
“Enable verbose compilation (--verbose)" is less clear than the surrounding command-style guidance because it omits the actual command. Consider using the full invocation (e.g., gh aw compile --verbose) for consistency and to reduce ambiguity.
| Enable verbose compilation (`--verbose`), set `ACTIONS_STEP_DEBUG = true`, inspect lock files, check MCP config (`gh aw mcp inspect`), and review logs. | |
| Enable verbose compilation (`gh aw compile --verbose`), set `ACTIONS_STEP_DEBUG = true`, inspect lock files, check MCP config (`gh aw mcp inspect`), and review logs. |
Summary
Reduced
common-issues.mdfrom 419 to 345 lines (18% reduction, -74 lines) while preserving all essential information.Changes Made
Consolidation improvements:
Structural improvements:
Maintained quality:
Impact
Before: 419 lines with repetitive patterns and verbose explanations
After: 345 lines with direct, scannable troubleshooting steps
Users can now find solutions faster with less scrolling and reading.
Testing Notes
Unable to capture screenshots due to Playwright localhost access restrictions (ERR_CONNECTION_REFUSED). The preview server was verified working via curl, but Playwright couldn't connect to localhost:4321 due to network isolation/sandboxing.
Files Changed
docs/src/content/docs/troubleshooting/common-issues.md(419 → 345 lines, -74 lines)Part of the documentation unbloat initiative to improve clarity and reduce cognitive load.