docs(skills): update queue-dashboard.md for Clanker Control Panel#397
docs(skills): update queue-dashboard.md for Clanker Control Panel#397castrojo wants to merge 2 commits 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
- Rename to Clanker Control Panel throughout - Add review-guide.md URL (machine-readable reviewer briefing) - Add data.json URL for machine consumption - Document approval count display on PR cards - Document 10-min cron cadence (was hourly) - Document data architecture (file cache, data.json, browser 5-min refresh) - Add theme toggle mention - Add pointer to agentic-contributing guide Assisted-by: Claude Sonnet 4.6 via GitHub Copilot
📝 WalkthroughWalkthroughThis PR introduces three new documentation pages under ChangesAgent skill documentation
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (3 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 |
dosubot
Bot
added
size:L
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (3)
docs/skills/queue-dashboard.md (2)
26-30: 💤 Low valueConsider adding language specifier to code block.
The fenced code block showing GitHub search filters would benefit from a language specifier for better rendering.
📝 Proposed 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> As per coding guidelines, markdownlint-cli2 flags fenced code blocks without a language specified (MD040). <details> <summary>🤖 Prompt for AI Agents</summary>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.mdaround lines 26 - 30, The fenced code block
containing the GitHub search filters (the lines starting with "is:pr is:open
-is:draft status:success review:...") is missing a language specifier and
triggers markdownlint MD040; update that code fence to include a language tag
(e.g., "text") so it becomestext ..., preserving the contents exactly
and leaving surrounding markdown unchanged.</details> --- `55-64`: _💤 Low value_ **Consider adding language specifier to code block.** The data architecture diagram would be clearer with a language specifier such as `text` or `mermaid`. <details> <summary>📝 Proposed fix</summary> ```diff -``` +```text GitHub API (every 10 min, GHA cron) → generate.js (25-min file TTL cache layer) → data.json ← committed to repo root, served as static asset ``` </details> As per coding guidelines, markdownlint-cli2 flags fenced code blocks without a language specified (MD040). <details> <summary>🤖 Prompt for AI Agents</summary>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.mdaround lines 55 - 64, The fenced code block
containing the architecture diagram (the block that starts with "GitHub API
(every 10 min, GHA cron)" and includes references like generate.js, data.json
and DATA) lacks a language specifier and triggers MD040; add an appropriate
language tag (e.g., text or mermaid) to the opening fence (changetotext
or ```mermaid) so markdownlint-cli2 passes while preserving the exact block
contents.</details> </blockquote></details> <details> <summary>docs/skills/e2e-ci.md (1)</summary><blockquote> `27-29`: _⚡ Quick win_ **Add language specifier to code block.** The fenced code block should specify a language for proper syntax highlighting and markdown compliance. <details> <summary>📝 Proposed fix</summary> ```diff -``` +```text systemd.mask=brew-setup.service ``` ``` </details> As per coding guidelines, markdownlint-cli2 flags fenced code blocks without a language specified (MD040). <details> <summary>🤖 Prompt for AI Agents</summary>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.mdaround lines 27 - 29, The fenced code block containing
"systemd.mask=brew-setup.service" lacks a language specifier; update that block
to include a language (e.g., "text" or "ini") so markdownlint MD040 is satisfied
and syntax highlighting is applied—locate the fenced block around the
"systemd.mask=brew-setup.service" line and change the opening fence from ``` to
🤖 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`:
- Line 7: Update the incorrect workflow path string
"common/.github/workflows/e2e.yml" in docs/skills/e2e-ci.md to reference the
actual repository location by replacing it with
"projectbluefin/common/.github/workflows/e2e.yml" so the documentation points to
the correct reusable workflow file.
In `@docs/skills/INDEX.md`:
- Line 7: The INDEX entry referencing hive-review.md in docs/skills/INDEX.md is
pointing to a non-existent file; either add the missing hive-review.md to the
repo or update the link text to the correct existing filename or path (e.g.,
replace "hive-review.md" with the actual filename/path that fd found) so the
relative link resolves; update the table row that currently contains
"[hive-review.md](hive-review.md)" accordingly.
---
Nitpick comments:
In `@docs/skills/e2e-ci.md`:
- Around line 27-29: The fenced code block containing
"systemd.mask=brew-setup.service" lacks a language specifier; update that block
to include a language (e.g., "text" or "ini") so markdownlint MD040 is satisfied
and syntax highlighting is applied—locate the fenced block around the
"systemd.mask=brew-setup.service" line and change the opening fence from ``` to
```text (or another appropriate language).
In `@docs/skills/queue-dashboard.md`:
- Around line 26-30: The fenced code block containing the GitHub search filters
(the lines starting with "is:pr is:open -is:draft status:success review:...") is
missing a language specifier and triggers markdownlint MD040; update that code
fence to include a language tag (e.g., "text") so it becomes ```text ... ```,
preserving the contents exactly and leaving surrounding markdown unchanged.
- Around line 55-64: The fenced code block containing the architecture diagram
(the block that starts with "GitHub API (every 10 min, GHA cron)" and includes
references like generate.js, data.json and __DATA__) lacks a language specifier
and triggers MD040; add an appropriate language tag (e.g., text or mermaid) to
the opening fence (change ``` to ```text or ```mermaid) so markdownlint-cli2
passes while preserving the exact block contents.
🪄 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: 464bf174-0208-4383-b6c2-f3ad6523f9a3
📒 Files selected for processing (3)
docs/skills/INDEX.mddocs/skills/e2e-ci.mddocs/skills/queue-dashboard.md
|
|
||
| ## How the common suite runs | ||
|
|
||
| The common post-merge E2E (`common/.github/workflows/e2e.yml`) calls the reusable |
There was a problem hiding this comment.
Correct the workflow path.
The path common/.github/workflows/e2e.yml is incorrect. The file is located at .github/workflows/e2e.yml within the projectbluefin/common repository. The common/ prefix should not be part of the path.
📝 Proposed fix
-The common post-merge E2E (`common/.github/workflows/e2e.yml`) calls the reusable
+The common post-merge E2E (`.github/workflows/e2e.yml`) calls the reusable📝 Committable suggestion
‼️ 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.
| The common post-merge E2E (`common/.github/workflows/e2e.yml`) calls the reusable | |
| The common post-merge E2E (`.github/workflows/e2e.yml`) calls the reusable |
🧰 Tools
🪛 LanguageTool
[uncategorized] ~7-~7: The official name of this software platform is spelled with a capital “H”.
Context: ... suite runs The common post-merge E2E (common/.github/workflows/e2e.yml) calls the reusable ...
(GITHUB)
🤖 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` at line 7, Update the incorrect workflow path string
"common/.github/workflows/e2e.yml" in docs/skills/e2e-ci.md to reference the
actual repository location by replacing it with
"projectbluefin/common/.github/workflows/e2e.yml" so the documentation points to
the correct reusable workflow file.
|
|
||
| | 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
# Check if hive-review.md exists in docs/skills/
fd -t f 'hive-review.md' docs/skills/Repository: projectbluefin/common
Length of output: 47
Fix broken link in docs/skills/INDEX.md to hive-review.md
fd -t f 'hive-review.md' docs/skills/ returns no matches in the current repo state, so the index entry’s hive-review.md relative link appears to be broken unless that file is added by this PR (or the link is adjusted).
🤖 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 referencing hive-review.md
in docs/skills/INDEX.md is pointing to a non-existent file; either add the
missing hive-review.md to the repo or update the link text to the correct
existing filename or path (e.g., replace "hive-review.md" with the actual
filename/path that fd found) so the relative link resolves; update the table row
that currently contains "[hive-review.md](hive-review.md)" accordingly.
1 participant
Updates
docs/skills/queue-dashboard.mdwith all changes from recent sessions:review-guide.mdURL — machine-readable reviewer briefingdata.jsonURL for machine/agent consumptiondata.json→ browser 5-min refresh)Note: this branch is based on #394 (
docs/session-e2e-queue-knowledge) which hasn't merged yet — it includesdocs/skills/INDEX.md,e2e-ci.md, andqueue-dashboard.mdfrom that PR. Consider merging #394 first or squashing together.Summary by CodeRabbit