Skip to content

chore: privacy, support and docs links for the plugin listing - #13

Merged
MarkAlex1234 merged 2 commits into
mainfrom
chore/plugin-links
Oct 6, 2026
Merged

MarkAlex1234 merged 2 commits into
mainfrom
chore/plugin-links

Conversation

@MarkAlex1234

Copy link
Copy Markdown
Member

Summary

Fills the 3 empty links the Claude directory listing shows:

 claude-plugin/.claude-plugin/plugin.json
+  "documentationUrl": ".../go-tokenless#readme"
+  "supportUrl":       ".../go-tokenless/issues"
+  "privacyPolicyUrl": ".../blob/main/PRIVACY.md"
+PRIVACY.md   no personal data and no telemetry; the only network use is read-only npm registry lookups (--offline turns them off)

Merge danger

Door: two-way

Blast radius: metadata and docs only.

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Docs and metadata only. I found no blocking issues.

  • The PRIVACY.md network claim matches the code. The only fetch is a GET to registry.npmjs.org/<pkg>, and --offline skips it (src/plan.ts:190-197).
  • The --read-token option only writes a ${{ secrets.NAME }} reference into the workflow. It never reads the secret value, so "Never accessed: secrets, tokens" holds.
  • I did not run the build or tests, and did not verify the plugin.json field names against Anthropic's directory schema.
  • The new URLs point at blob/main/PRIVACY.md, which only resolves once this PR is merged.

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Docs and metadata only. PRIVACY.md matches the code: the only network call is the read-only GET to registry.npmjs.org in src/plan.ts:194. The plugin.json URLs look right. I did not run the build or tests.

One non-blocking note: the privacyPolicyUrl points at blob/main/PRIVACY.md, so it returns 404 until this PR is merged. Submit the directory listing after the merge.

@MarkAlex1234
MarkAlex1234 merged commit 66a25f3 into main Oct 6, 2026
6 checks passed
@MarkAlex1234
MarkAlex1234 deleted the chore/plugin-links branch October 6, 2026 08:27
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.

1 participant