Repository navigation
chore: archive pal-health and session-analytics plugins - #528
Merged
Merged
Conversation
pal-health is disabled in every settings.json and duplicates nothing worth keeping standalone. session-analytics is replaced by the token-meter MCP server; AGENTS.md now points there. process-cleanup stays: upstream anthropics/claude-code#1935 is still open. Assisted-by: Claude:claude-sonnet-5 Claude-Session: https://claude.ai/code/session_01CaZtxKWkgukysSGrCq8Tmz
Jacob (JacobPEvans-personal)
added a commit
that referenced
this pull request
Sep 22, 2026
…p the exclude Claude Code's paths: frontmatter field accepts either a YAML list or a comma-separated string (code.claude.com/docs/en/memory#path-specific-rules, rule frontmatter reference). cclint's rules_linter only accepts the comma-separated string form and fails a YAML list with "paths: field must be a string" — even though Claude Code reads the list form correctly. Switch both files to the string form so Claude Code and cclint agree, and remove the .cclintrc.yaml exclude added in the previous commit — excluding a check is not a fix. Reproduced the original failure against PR #528's diff (git reset --soft to its parent, restage, run cclint --staged directly against the pinned binary) and confirmed it's resolved by this change alone, with no exclude in place. Assisted-by: Claude:claude-sonnet-5 Claude-Session: https://claude.ai/code/session_01CaZtxKWkgukysSGrCq8Tmz
3 tasks done
# Conflicts: # .claude-plugin/categories/core.nix # .claude-plugin/marketplace.json # .release-please-manifest.json # AGENTS.md # release-please-config.json
openbao-service-broker Bot
added a commit
that referenced
this pull request
Sep 22, 2026
…gs (#534) * fix: exclude .claude/rules/ from cclint's paths: validator cclint v0.57.0's rules_linter requires the paths: frontmatter field to be a single comma-separated string and fails every .claude/rules/*.md file that uses a YAML list with "paths: field must be a string" — whenever any plugin.json/marketplace.json diff triggers a full project scan (confirmed: reproduces on PR #528's diff, which touches neither rule file). Anthropic's own docs (code.claude.com/docs/en/memory#path-specific-rules) document paths: as a YAML list, and Claude Code only reads it that way (confirmed by the fix/rules-paths-key change, 3b39f38). Our two rule files are correct; cclint's validator predates or diverges from that spec. Filed upstream: dotcommander/cclint#3 Excluding .claude/rules/** from cclint's scope isn't suppressing a real defect — it's correctly scoping a third-party tool away from a check it gets wrong for files it wasn't built to validate against the current spec. Assisted-by: Claude:claude-sonnet-5 Claude-Session: https://claude.ai/code/session_01CaZtxKWkgukysSGrCq8Tmz * fix: resolve pre-existing shellcheck findings in guard scripts - write-script-guard.sh: drop the dead */.claude/plugins/* pattern (SC2222), unreachable because the earlier */plugins/* pattern already matches it (SC2221); replace the sed strip with bash parameter expansion (SC2001). - validate-markdown.sh: quote the ${lint_dir} expansion inside the parameter-expansion pattern so it can't be read as a glob (SC2295). Assisted-by: Claude:claude-sonnet-5 Claude-Session: https://claude.ai/code/session_01CaZtxKWkgukysSGrCq8Tmz * fix(rules): use the comma-separated paths: form cclint validates, drop the exclude Claude Code's paths: frontmatter field accepts either a YAML list or a comma-separated string (code.claude.com/docs/en/memory#path-specific-rules, rule frontmatter reference). cclint's rules_linter only accepts the comma-separated string form and fails a YAML list with "paths: field must be a string" — even though Claude Code reads the list form correctly. Switch both files to the string form so Claude Code and cclint agree, and remove the .cclintrc.yaml exclude added in the previous commit — excluding a check is not a fix. Reproduced the original failure against PR #528's diff (git reset --soft to its parent, restage, run cclint --staged directly against the pinned binary) and confirmed it's resolved by this change alone, with no exclude in place. Assisted-by: Claude:claude-sonnet-5 Claude-Session: https://claude.ai/code/session_01CaZtxKWkgukysSGrCq8Tmz --------- Co-authored-by: JacobPEvans <20714140+JacobPEvans-personal@users.noreply.github.com>
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.
Summary
pal-health(disabled in every settings.json; no unique value)session-analytics, replaced by thetoken-meterMCP server;AGENTS.mdnow points thereprocess-cleanup— upstream [BUG] MCP servers not properly terminated when Claude Code exits, causing orphaned processes anthropics/claude-code#1935 (the bug it works around) is still openTest plan
🤖 Generated with Claude Code
https://claude.ai/code/session_01CaZtxKWkgukysSGrCq8Tmz