Skip to content

feat: tag codex harness on signup + plugin-setup prompt#5

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

feat: tag codex harness on signup + plugin-setup prompt#5
dimavrem22 merged 9 commits into
mainfrom
feat/agent-signup-harness

Conversation

@dimavrem22

@dimavrem22 dimavrem22 commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

What

Two small changes:

  1. Signup attribution — pass harness="codex" as a keyword to the Inkbox.signup(...) call in inkbox_codex/setup_wizard.py so agents that self-sign-up from this harness are tagged accordingly.
  2. Plugin-permission prompt — add a short note to CHANNEL_PROMPT (inkbox_codex/prompts.py) telling the agent that if it self-signed-up to Inkbox without this plugin configured, it should ask the human for permission before setting up the plugin, since the plugin wires native messaging channels directly into its runtime.

Dependency

The harness keyword depends on the Inkbox Python SDK gaining a harness parameter on Inkbox.signup(...). This PR should land alongside / after that SDK change.

Related PRs (agent-signup harness)

Pass harness="codex" on Inkbox.signup so self-signups are attributed to
this harness, and add a channel-prompt note telling the agent to ask the
human for permission before configuring the plugin when it self-signed up
without it already set up.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@dimavrem22

Copy link
Copy Markdown
Contributor Author

codex: This now passes harness="codex" 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.

The wizard now passes harness="codex" to Inkbox.signup, which needs SDK
0.4.9. Bump the pin everywhere (pyproject, INKBOX_REQUIREMENTS, doctor,
gateway, tests) and add a minimum-version guard in _ensure_inkbox_sdk so
an importable-but-stale inkbox takes the existing install/upgrade path.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@dimavrem22

Copy link
Copy Markdown
Contributor Author

Fixed in f7ab3b0: bumped inkbox>=0.4.7>=0.4.9 (pyproject, INKBOX_REQUIREMENTS, hints, requirement-string tests) and added a MIN_INKBOX_VERSION guard in _ensure_inkbox_sdk() so an importable-but-stale (<0.4.9) SDK is treated as missing and upgraded via the existing install path before the harness= call.

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

Copy link
Copy Markdown
Contributor Author

Correction: the harness-capable SDK release is 0.4.10, not 0.4.9. Pins, INKBOX_REQUIREMENTS, the min-version guard, hints, and tests are now all 0.4.10 (c65fd5b).

dimavrem22 and others added 5 commits June 25, 2026 05:10
Rework the setup-guidance copy so the agent confirms with its human and
sets up the plugin together, never starting on its own.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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>
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>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@dimavrem22 dimavrem22 marked this pull request as ready for review June 26, 2026 20:15
@dimavrem22 dimavrem22 merged commit 9e1684d into main Jun 26, 2026
1 check passed
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