Require hosting intent for here.now skill selection - #9
Open
wilbertpariguana wants to merge 1 commit into
Open
wilbertpariguana wants to merge 1 commit into
wilbertpariguana wants to merge 1 commit into
Conversation
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.
The description currently includes generic webpage and chatbot creation as standalone triggers. A localhost-only prototype can therefore match a hosting skill's selection text. This makes hosting/live-sharing intent explicit and excludes local-only creation or another named host, while preserving explicit here-now invocation and management of hosted sites/files.
Inspected versions: installed 1.27.0 and current public 1.28.0 at
e139365a152a4617d168c3be17c9c1486b67e39b; both retain the broad selector. This patch changes only the canonical description and its generated compatibility mirror. The two files remain byte-identical; workflow bodies, scripts, authentication requirements and install commands are preserved.Synthetic source-review examples (not executed routing trials):
$here-nowto explain private-site options without changing anything.Related: #8 addresses Hermes index truncation by shortening metadata and relocating the original triggers into a
When to Usesection, including local page/chatbot creation. This proposal addresses a distinct selection boundary and does not depend on a character budget. Both touch the description; the hosting-intent boundary should be retained when reconciling them. #5 addresses runtime metadata and credential handling, which this patch does not change.Per AGENTS.md,
here-now/is canonical andskills/here-now/is its compatibility mirror. The public repo is synced from the private product source; please carry this canonical correction into that source and regenerate the applicable distribution mirrors so a later sync retains it. The Hermes-specific bundle is outside this two-file proposal.Validation: clean patch application to current upstream; applied bytes match the independently reviewed proposal; body equality, canonical/mirror equality, YAML and generic skill validation passed. The four public version/skill discovery endpoints required by AGENTS.md returned HTTP 200. Independent source review passed and is reused unchanged.
Runtime behavior was not tested. Automatic misselection and publication were not observed. Synthetic examples express expected selection from source review, not executed behavioral tests or measured improvements. No publishing, account or connected workflow was run.