Skip to content

feat(scanner): reject plugin dart and flutter pub publish - #1184

Draft
seonghobae wants to merge 9 commits into
feat/claude-plugin-gem-nuget-1099from
feat/claude-plugin-pub-publish-1099
Draft

feat(scanner): reject plugin dart and flutter pub publish#1184
seonghobae wants to merge 9 commits into
feat/claude-plugin-gem-nuget-1099from
feat/claude-plugin-pub-publish-1099

Conversation

@seonghobae

@seonghobae seonghobae commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Successor of #1183 / issue #1099. Does not Close those. gem/nuget stay #1183. pnpm/uv/poetry stay #1182. Pass is not Noema admission.

Unique delta

Fail closed when a plugin hook or manifest runs remaining Dart registry writes that #1183 left as inventory:

  • dart pub publish, flutter pub publish, and legacy pub publish as claude-plugin-pub-publish-command (CWE-269)
  • dart pub get and flutter pub get stay inventory
  • pnpm publish stays claude-plugin-pnpm-publish-command
  • unquoted # comments, echo/printf lookalikes, and marketplace description prose are not this class
  • README pub wording is not a hook command
  • snippets are command labels, not tokens or secrets
  • composer publish and go upload are not invented; they are not real package-manager CLIs

#1183 remains owner of gem/nuget push.

Evidence

  • RED 4351d47 then GREEN 09b6dce
  • tests/test_claude_plugin_pub_publish.py plus sibling gem/alt-publish lock
  • plugin coverage suite 506 passed on Python 3.13

Relates to #1099. Relates to #1183.

Current assignment-value context repair — 2026-09-08

This normal two-parent descendant inherits canonical #1173 RED 90ae232e852b3d01463d978f9bccd9157c341fe7 and GREEN 1e6a3eb43a5bc2d146656af13e7fa6cad2a77315 without a leaf workaround. Exact head: 3b5f6416c5cba8f6ce16c53c0e6b74575448e059; current parent: 3e571723a65266c086912b9f3bf1de648150cc01; Git compare is 0 behind and preserves this PR's unique detector/test delta. Assignment-value negatives and environment-prefix/direct/substitution positives are present in the exact tree. This custom-base PR remains Draft until its own hosted integration and independent current-head review are terminal. Earlier head strings in this body are historical evidence only.

Current heredoc-safe restack — 2026-09-08

Canonical #1173 exact head 184b4b4f55906284ac1d8059e46cec8f0d9cfc9a fixes closed literal here-document payload false positives with RED 6f6be77d... → GREEN 12faf2f.... This branch was integrated non-destructively on parent 4253ddeb380245db514c885fc6bf26662515211c; exact head is bde44516021aa6c1fd6c2be7c7beb9345a671d97. GitHub compare reports behind_by=0, ahead_by=9; the PR's unique detector delta remains in the tree. Exact-head hosted integration and independent review are not terminal, so this PR remains Draft. Earlier head strings are historical evidence only.

Hook and manifest dart pub publish, flutter pub publish, and legacy
pub publish must fail closed. pub get, comments, echo lookalikes,
and README wording stay inventory. Relates to #1099.
Fail closed on executable dart pub publish, flutter pub publish, and
legacy pub publish. dart pub get and comments stay inventory.
Relates to #1099.
@coderabbitai

coderabbitai Bot commented Sep 8, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

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.

seonghobae added a commit that referenced this pull request Sep 8, 2026
Exact head 7faaaad stacked on #1183 fails closed on dart pub
publish, flutter pub publish, and legacy pub publish. Relates to
#1099.
seonghobae added a commit that referenced this pull request Sep 8, 2026
Merge concurrent #999 quoted-command-context snapshot. Exact head
7faaaad stacked on #1183 fails closed on dart/flutter pub publish.
Relates to #1099.
seonghobae added a commit that referenced this pull request Sep 8, 2026
Exact head 512710e stacked on #1184 fails closed on hex publish,
mix hex.publish, conda upload, and anaconda upload. Relates to #1099.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request priority: high

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

1 participant