Skip to content

Report Whether the Fleet Skills Plugin Is Installed and Enabled in the Live Channel - #2701

Merged
ptr727 merged 5 commits into
developfrom
feature/auto-1757
Oct 10, 2026
Merged

ptr727 merged 5 commits into
developfrom
feature/auto-1757

Conversation

@ptr727

@ptr727 ptr727 commented Oct 10, 2026

Copy link
Copy Markdown
Owner

skills_install.py --report now carries live.plugin with installed and enabled, read from claude plugin list --json for the user-scope fleet-skills@projecttemplate-fleet entry (null where the listing cannot be read), only when the marketplace is registered. A shared claude_json helper backs both listings. The check-this-repo skill and scripts/README.md name the field.

Closes on promotion: #1757

Remaining local-review findings after the two-round edit budget (class introduced):

  • enabled is read from the directory --report runs in, so a project-level enabledPlugins override in that directory changes it. The skill and README present it as machine-wide.
  • A project- or local-scope-only install reads installed: false, which the skill and README word as "serves nothing".
  • No test asserts claude_json passes timeout=SUBPROCESS_TIMEOUT (small).

Not changed: Windows claude.cmd resolution via bare claude (pre-existing in marketplace_entry).

🤖 Generated with Claude Code

ptr727 and others added 3 commits October 9, 2026 23:07
…e Live Channel

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings October 10, 2026 06:11
@coderabbitai

coderabbitai Bot commented Oct 10, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Repository: ptr727/ProjectTemplate/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: efc188f3-c9f5-4a18-a4d8-c453be106325

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

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

@codecov

codecov Bot commented Oct 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (develop@d54da4c). Learn more about missing BASE report.

Additional details and impacted files
@@            Coverage Diff             @@
##             develop    #2701   +/-   ##
==========================================
  Coverage           ?   59.67%           
==========================================
  Files              ?       16           
  Lines              ?     8334           
  Branches           ?        0           
==========================================
  Hits               ?     4973           
  Misses             ?     3361           
  Partials           ?        0           
Flag Coverage Δ
python-3.13 59.67% <100.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

Plugin state can be misreported across scopes and working directories, and timeout coverage is missing.

4 open findings
What changed in this PR

Adds live Claude plugin installation and enabled-state reporting to skills_install.py, with tests and documentation updates.

Changes:

  • Adds shared Claude JSON querying and plugin-state reporting.
  • Extends tests for plugin states and live-channel output.
  • Updates skill documentation and generated distributions.
File Description
tests/​test_skills_install.py Tests plugin-state and live-channel behavior.
scripts/​skills_install.py Reports plugin installation and enablement state.
scripts/​README.md Documents the new report fields.
.github/​skills/​check-this-repo/​SKILL.md Updates generated GitHub skill guidance.
.claude-plugin/​fleet-skills/​skills/​check-this-repo/​SKILL.md Updates generated Claude skill guidance.
.claude-plugin/​fleet-skills/​.source-digests/​check-this-repo Refreshes the source digest.
.agents/​skills/​check-this-repo/​SKILL.md Updates canonical plugin health guidance.

🧠 Review effort: Lite


💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread scripts/skills_install.py
Comment thread .agents/skills/check-this-repo/SKILL.md Outdated
Comment thread scripts/README.md Outdated
Comment thread scripts/skills_install.py
ptr727 and others added 2 commits October 9, 2026 23:17
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@ptr727

ptr727 commented Oct 10, 2026

Copy link
Copy Markdown
Owner Author

A recorded local strict-review pass covers head 2f23ab03eac6e20fecb7bc4d670e3486aa536069, the content this pull request carries at that commit against develop, and it recorded 4 findings.

@ptr727
ptr727 merged commit 4774eac into develop Oct 10, 2026
13 checks passed
ptr727 added a commit that referenced this pull request Oct 10, 2026
… Corrections to Main (#2740)

## Summary

Promotes 20 changes from `develop` to `main`:

- #2737: Flag a Lone Semicolon After an Explanatory Colon in the Prose
Gate
- #2734: Share pr_review.py wait's Liveness Readings and Open the Held
Poll on Its Snapshot
- #2732: Share One Bounded Backoff Loop Between pr_review.py wait's Two
Polls
- #2729: Document pr_review.py wait's Immediate 44 and check_nodes's
Node Keys
- #2727: Narrow pr_review.py wait's Exit 44 to Required Checks
- #2722: Poll an Attested Head's Checks in pr_review.py wait
- #2720: Name the Command That Enumerates Open Feature Pull Requests in
backlog-burndown
- #2718: Point the Skills Refresh Cadence at host-setup.md
- #2716: Fall Back to os.defpath for PATH in Two Test Harnesses
- #2714: State the Pip Form's Root-Config Type-Check Command in
python-codestyle
- #2712: State the Build Profile's CI Type Check as the Validator Runs
It
- #2709: Bring the Fleet-Map workflow-ci-contract Entry and G9 Gap Up to
the Skill Description
- #2707: Bring the Line-Endings Reference and a Test Docstring to the
Corrected Wording
- #2705: Drop the Stale Utilities driftNote From the Registry
- #2703: Remove the Inert SC2016 Directives in configure.sh and Correct
the shell-codestyle Claim
- #2701: Report Whether the Fleet Skills Plugin Is Installed and Enabled
in the Live Channel
- #2699: Align the Audit Report Template Dimensions With AUDIT.md
Section 4
- #2697: Harden the Source-Pinning Assertions in test_pr_review.py
- #2695: Name the Off-Grammar --branch Outcome in AUDIT.md
- #2693: Count Every Unresolved Review Thread in pr_review.py

Closes #1396
Closes #2731
Closes #2723
Closes #2725
Closes #2724
Closes #2685
Closes #1308
Closes #2191
Closes #1862
Closes #2711
Closes #2025
Closes #1243
Closes #1237
Closes #1115
Closes #1156
Closes #1757
Closes #1593
Closes #1732
Closes #1509
Closes #1404

🤖 Generated with [Claude Code](https://claude.com/claude-code)
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