Repository navigation
fix(plugin): privacy policy links, and keep the install command out of the hook - #358
Merged
Merged
Conversation
…f the hook - plugin.json: privacyPolicyUrl and termsOfServiceUrl, which the plugin directory shows on the listing. The submission form asks you to confirm the plugin's privacy policy, and none was linked. - hooks/check-cli.sh: stop printing the curl | sh install command. The directory's validator flagged the hook under "Scripts the validator couldn't follow", most likely because the hook's text contains an installer command. The hook now points Claude to the install section of the skill, which already has the command, and still asks the user first.
eddietejeda
requested review from
rohan-hotdata
and removed request for
a team
October 18, 2026 21:31
| rest of this session. | ||
| user and ask whether to install it. If they agree, follow the install steps in | ||
| the hotdata skill (the "Upgrade" section of skills/hotdata/SKILL.md), and set | ||
| HOTDATA_NO_SKILLS=1 because this plugin already provides the skill. |
Contributor
There was a problem hiding this comment.
nit: Restore the instruction to call the binary as ~/.hotdata/cli/hotdata after an install (not blocking).
The hook adds ~/.hotdata/cli to PATH only at session start. A prebuilt install during the session leaves hotdata off PATH until the next session. The removed text covered this case. The "Upgrade" section of skills/hotdata/SKILL.md (lines 492-498) does not cover it. Without the instruction, Claude runs hotdata and gets "command not found" right after a successful install.
Suggested replacement for the message body:
HOTDATA_NO_SKILLS=1 because this plugin already provides the skill. If the
binary lands in ~/.hotdata/cli, call it as ~/.hotdata/cli/hotdata for the rest
of this session.
This text has no curl or sh, so the validator change still holds.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
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.
Follow-up from the plugin directory submission's Compliance step.
privacyPolicyUrl(https://www.hotdata.dev/privacy-policy) andtermsOfServiceUrl(https://www.hotdata.dev/terms-of-service) toplugin.json. The directory shows both on the listing. The form asks you to confirm "the plugin's privacy policy accurately describes the data it handles", and until now no policy was linked.curl … | shinstall command. It tells Claude to follow the install section in the skill, which already has the command, and to setHOTDATA_NO_SKILLS=1. Nothing changes for users: Claude still asks before installing anything. This is a guess at the cause, since the portal didn't name the file.Not addressed: the "Image or font file that the plugin's code could run" hold. The only image is the listing icon, and the only reference to it is the
iconfield the portal asked for. Removing it would bring back the "No icon" warning, so it stays until the hold's details show what the validator wants.Verified
claude plugin validate --strictpasses.shellcheckis clean.curl/sh.check-skills.sh --require-versionpasses.Privacy policy gap for the policy owner: the current policy covers account and website data. It doesn't mention customer data loaded into Hotdata, and it lists "analytics and telemetry partners" while the README says the CLI collects no telemetry. Those can both be true, but a reviewer may ask.