Skip to content

docs: OpenCode plugin live-verification learning#192

Merged
marcusrbrown merged 1 commit into
mainfrom
docs/opencode-plugin-live-verify-learning
Jul 10, 2026
Merged

docs: OpenCode plugin live-verification learning#192
marcusrbrown merged 1 commit into
mainfrom
docs/opencode-plugin-live-verify-learning

Conversation

@marcusrbrown

Copy link
Copy Markdown
Collaborator

Captures the durable lesson from building the OpenCode Impeccable plugin (#191): an OpenCode plugin's real contract — the host loader's export semantics and the subprocess env/stdin/stdout contract — lives outside the unit-test boundary. Four distinct silent-failure bugs each passed the full unit suite, tsc, and lint clean, and were caught only by live-session verification, one restart cycle each.

The doc records the verification protocol that would have caught them sooner: restart after any plugin change, confirm the plugin actually loaded (grep the host log), exercise the real path, cross-check side-effect evidence (subprocess cache mtime) rather than just the visible result, keep the plugin module to a single function export, probe the subprocess boundary directly to isolate bugs without a restart per hypothesis, and design a fail-loud path that can actually fire when the subprocess always exits 0.

Cross-linked into the existing "local green ≠ production truth" cluster (unit-green, pwa-service-worker, dev-server-hang, fixture-harness). Docs-only — no runtime or release impact.

@fro-bot fro-bot added the agent: working Agent is currently working on this label Jul 10, 2026

@fro-bot fro-bot left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Verdict: PASS

Docs-only addition (docs/solutions/workflow-issues/opencode-plugin-hooks-need-live-verification-2026-07-10.md, +128/-0). No code, config, or CI changes — verified via gh pr diff.

Blocking issues

None

Non-blocking concerns

None

Missing tests

Not applicable — documentation-only change with no runtime or release impact.

Risk assessment

LOW — this is a solutions-doc entry with no code, dependency, auth, or workflow changes. Verified:

  • All four cross-linked docs in the "Related" section exist in the repo (unit-green-is-not-feature-done..., pwa-service-worker-registration-invisible-to-unit-tests..., dev-server-hang-background-no-watch-kill-orphans..., local-fixture-harness-must-mirror-wire-contract...).
  • Frontmatter shape (category, problem_type, applies_when, tags) matches conventions used elsewhere in docs/solutions/workflow-issues/.
  • No secrets, tokens, or credentials present in the file.

Run Summary
Field Value
Event pull_request
Repository fro-bot/dashboard
Run ID 29131030260
Cache hit
Session ses_0b194e51bffeqE57BUGnx02QR0

@fro-bot fro-bot removed the agent: working Agent is currently working on this label Jul 10, 2026
@marcusrbrown marcusrbrown merged commit 31b458f into main Jul 10, 2026
7 checks passed
@marcusrbrown marcusrbrown deleted the docs/opencode-plugin-live-verify-learning branch July 10, 2026 23:49
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.

2 participants