Skip to content

feat: tag hermes harness on signup + plugin-setup skill - #19

Merged
dimavrem22 merged 9 commits into
mainfrom
feat/agent-signup-harness
Jun 26, 2026
Merged

feat: tag hermes harness on signup + plugin-setup skill#19
dimavrem22 merged 9 commits into
mainfrom
feat/agent-signup-harness

Conversation

@dimavrem22

@dimavrem22 dimavrem22 commented Jun 25, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Two small changes:

  • Signup harness tag: pass harness="hermes" to Inkbox.signup(...) in setup_wizard.py so the backend records which harness drove a self-signup.
  • Plugin-setup skill: add skills/inkbox-plugin-setup/SKILL.md instructing an agent that self-signed up to Inkbox without this plugin configured to ask the human for permission before setting up the plugin, since doing so wires native messaging channels into the agent's runtime.

Depends on the Inkbox SDK gaining the harness keyword param on signup(...). Until that lands, this call will fail on the unexpected kwarg.

Related PRs (agent-signup harness)

This was referenced Jun 25, 2026
Merged
Merged
Merged
Merged
@dimavrem22

dimavrem22 commented Jun 25, 2026

Copy link
Copy Markdown
Contributor Author

codex: This now passes harness="hermes" into Inkbox.signup, but the plugin still declares/installs inkbox>=0.4.7, and _ensure_inkbox_sdk() returns immediately when any older SDK imports successfully. Since the harness keyword is introduced by the SDK PR (0.4.9), existing environments with inkbox==0.4.7 can raise TypeError before the request reaches the server. Please bump the dependency/bootstrap minimum to the SDK version that supports harness, or feature-detect the signup signature and fall back cleanly.

@dimavrem22

dimavrem22 commented Jun 25, 2026

Copy link
Copy Markdown
Contributor Author

Fixed in e5811a4: bumped inkbox>=0.4.7 → >=0.4.9 (pyproject, INKBOX_REQUIREMENTS, README, requirement-string tests) and added a min-version guard in _ensure_inkbox_sdk() so an importable-but-stale (<0.4.9) SDK is treated as missing and upgraded before the harness= call. All setup-wizard tests pass.

@dimavrem22

dimavrem22 commented Jun 25, 2026

Copy link
Copy Markdown
Contributor Author

Correction: the harness-capable SDK release is 0.4.10, not 0.4.9. Pins, INKBOX_MIN_VERSION, README, and tests are now all 0.4.10 (bcd00f6).

@dimavrem22
dimavrem22 marked this pull request as ready for review June 26, 2026 20:15
dimavrem22 and others added 9 commits June 26, 2026 21:12
@dimavrem22 @claude
feat: tag hermes harness on signup + plugin-setup skill
987fcb3 
Pass harness="hermes" to Inkbox.signup so the backend records which
harness drove a self-signup. Add a slim inkbox-plugin-setup skill that
tells an agent which self-signed-up without this plugin configured to
ask the human for permission before setting up the plugin, since it
grants native messaging channels in the agent's runtime.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@dimavrem22 @claude
chore: align harness plugin-setup copy with the shared canonical wording
5f4a366 
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@dimavrem22 @claude
Require inkbox>=0.4.9 and upgrade outdated SDK in setup
8b8e86c 
The plugin now passes harness="hermes" to Inkbox.signup, which needs SDK
0.4.9. Bump the pin everywhere and add a minimum-version guard in
_ensure_inkbox_sdk so an env with an older inkbox is treated like a missing
requirement and upgraded via the existing install path.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@dimavrem22 @claude
Bump inkbox SDK pin to 0.4.10 for agent-signup harness
36370b3 
0.4.10 is the release that introduces the agent-signup harness param;
0.4.9 was an earlier release that predates it.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@dimavrem22 @claude
Require confirming with human and setting up together before Inkbox p…
00012f2 
…lugin setup

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@dimavrem22 @claude
Fix inkbox-plugin-setup skill: agent may set up itself after asking p…
3c2386d 
…ermission

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@dimavrem22 @claude
chore: drop redundant inkbox-plugin-setup skill
14860b0 
The plugin's setup wizard already configures the plugin during signup, so a
separate agent-facing "ask permission, then set it up yourself" instruction is
redundant. Drop it; keep the harness= signup tagging.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@dimavrem22 @claude
ci: run pytest on PRs
e92baaf 
Mirror openclaw-plugin's CI. The inkbox SDK is deliberately not installed
(tests mock it, and inkbox>=0.4.10 with the harness= kwarg isn't on PyPI yet),
so CI installs only the deps the tests actually import.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@dimavrem22 @claude
chore: trim out-of-place harness/version comments
40e8d18 
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@dimavrem22
dimavrem22 force-pushed the feat/agent-signup-harness branch from ed81b34 to 40e8d18 Compare June 26, 2026 21:17
@dimavrem22
dimavrem22 merged commit 3027997 into main Jun 26, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@dimavrem22