docs(skills): add E2E CI architecture, queue dashboard, and skill INDEX#394
docs(skills): add E2E CI architecture, queue dashboard, and skill INDEX#394castrojo wants to merge 1 commit into
Conversation
Documents knowledge captured during 2026-06-02 review session: - e2e-ci.md: common suite SSH model, brew tools masked in CI, MOTD fix (testsuite PR #208), quarantined scenarios (zsh/fish/brew-tools, issue #210) - queue-dashboard.md: queue.projectbluefin.io PR tiers, common merge ruleset (2 approvals required, only Build+push is a required check) - INDEX.md: skill docs index Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Assisted-by: Claude Sonnet 4.6 via GitHub Copilot
dosubot
Bot
added
size:L
📝 WalkthroughWalkthroughThis pull request adds three new skill documentation files to establish a knowledge base for the projectbluefin/common repository. An index page provides entry-point navigation. Detailed documentation covers the E2E CI suite workflow, kernel configuration, test failures, and SSH execution model. A queue dashboard reference explains PR triage, merge requirements, and refresh behavior. ChangesSkills Documentation
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/skills/e2e-ci.md`:
- Around line 27-29: The fenced code block containing
"systemd.mask=brew-setup.service" is missing a language hint (MD040); update
that markdown block in docs/skills/e2e-ci.md by adding a fence language (e.g.,
use "text") so the block becomes ```text ... ``` and satisfies the markdown
linter while preserving the exact inner content.
In `@docs/skills/INDEX.md`:
- Line 7: The INDEX entry in docs/skills/INDEX.md points to hive-review.md which
doesn't exist; fix by either adding the missing docs/skills/hive-review.md file
with the intended content or updating the link in docs/skills/INDEX.md to the
correct existing document path/title (ensure the target filename
`hive-review.md` or its replacement is present in docs/ so the link is not dead
and docs build/links pass).
In `@docs/skills/queue-dashboard.md`:
- Around line 18-22: The code fence in docs/skills/queue-dashboard.md is missing
a language tag which triggers MD040; update the triple-backtick fence that
contains the PR query examples so it starts with ```text (i.e., change the
opening ``` to ```text) to explicitly mark the block as plain text and satisfy
the linter while keeping the content unchanged.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 31568ca5-5fea-4ad9-92be-87997f052ee8
📒 Files selected for processing (3)
docs/skills/INDEX.mddocs/skills/e2e-ci.mddocs/skills/queue-dashboard.md
| ``` | ||
| systemd.mask=brew-setup.service | ||
| ``` |
There was a problem hiding this comment.
Add a language hint to the fenced code block.
This block is missing a fence language (MD040). Use text (or another appropriate lexer) to satisfy markdown linting.
Suggested fix
-```
+```text
systemd.mask=brew-setup.service</details>
<!-- suggestion_start -->
<details>
<summary>📝 Committable suggestion</summary>
> ‼️ **IMPORTANT**
> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
```suggestion
🧰 Tools
🪛 markdownlint-cli2 (0.22.1)
[warning] 27-27: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@docs/skills/e2e-ci.md` around lines 27 - 29, The fenced code block containing
"systemd.mask=brew-setup.service" is missing a language hint (MD040); update
that markdown block in docs/skills/e2e-ci.md by adding a fence language (e.g.,
use "text") so the block becomes ```text ... ``` and satisfies the markdown
linter while preserving the exact inner content.
|
|
||
| | File | What it covers | | ||
| |---|---| | ||
| | [hive-review.md](hive-review.md) | `~/src/hive-status` — session start, P0/P1 triage, hive label taxonomy | |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Verify whether the linked docs file exists in the repository.
fd -i '^hive-review\.md$' docsRepository: projectbluefin/common
Length of output: 47
Fix dead docs link: hive-review.md is missing under docs/
The docs/skills/INDEX.md entry links to hive-review.md, but no hive-review.md file is present anywhere in docs/ (the link target appears to be a dead reference).
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@docs/skills/INDEX.md` at line 7, The INDEX entry in docs/skills/INDEX.md
points to hive-review.md which doesn't exist; fix by either adding the missing
docs/skills/hive-review.md file with the intended content or updating the link
in docs/skills/INDEX.md to the correct existing document path/title (ensure the
target filename `hive-review.md` or its replacement is present in docs/ so the
link is not dead and docs build/links pass).
| ``` | ||
| is:pr is:open -is:draft status:success review:approved → ✅ Approved (ready to merge) | ||
| is:pr is:open -is:draft status:success review:required → ⚠️ Needs reviews | ||
| is:pr is:open -is:draft status:success review:none → 🔵 No reviews yet | ||
| ``` |
There was a problem hiding this comment.
Specify a language for the query filter code fence.
This fence triggers MD040 because it has no language. text is sufficient here.
Suggested fix
-```
+```text
is:pr is:open -is:draft status:success review:approved → ✅ Approved (ready to merge)
is:pr is:open -is:draft status:success review:required → ⚠️ Needs reviews
is:pr is:open -is:draft status:success review:none → 🔵 No reviews yet</details>
<!-- suggestion_start -->
<details>
<summary>📝 Committable suggestion</summary>
> ‼️ **IMPORTANT**
> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
```suggestion
🧰 Tools
🪛 markdownlint-cli2 (0.22.1)
[warning] 18-18: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@docs/skills/queue-dashboard.md` around lines 18 - 22, The code fence in
docs/skills/queue-dashboard.md is missing a language tag which triggers MD040;
update the triple-backtick fence that contains the PR query examples so it
starts with ```text (i.e., change the opening ``` to ```text) to explicitly mark
the block as plain text and satisfy the linter while keeping the content
unchanged.
hanthor
left a comment
hanthor
left a comment
There was a problem hiding this comment.
queue-dashboard.md names the sync workflow as hive-progress-sync.yml, but the workflow being added by the related hive PRs is hive-status-sync.yml. Since these docs are intended as agent-facing reference, the wrong filename will cause lookup failures. Please correct the workflow name.
2 participants
What
Captures session knowledge as skill docs for future agents.
docs/skills/e2e-ci.md
Common suite E2E CI architecture:
bash -lcSSH commands (PATH mystery, tracked in testsuite#210)bash -lc+ login shell side effects)docs/skills/queue-dashboard.md
docs/skills/INDEX.md
Top-level index so agents can discover available skill docs at a glance.
Why
These were the primary knowledge gaps that caused wasted diagnostic time during
the 2026-06-02 review session. Future agents should be able to triage nightly
E2E failures and PR merge state without re-deriving this information.
Summary by CodeRabbit