Skip to content

fix(boatstack): append generic execution-mode notice to all agent templates - #71

Merged
bigboateng merged 1 commit into
mainfrom
fix/generic-execution-mode-dx
Jul 20, 2026
Merged

fix(boatstack): append generic execution-mode notice to all agent templates#71
bigboateng merged 1 commit into
mainfrom
fix/generic-execution-mode-dx

Conversation

@bigboateng

Copy link
Copy Markdown
Contributor

Reviewer-ready change

Why this change

Improve the Developer Experience (DX) for Gemini CLI, Claude Code, Cursor, and Codex when attempting to run Boatstack commands while operating in a restricted execution environment (like Gemini's Plan Mode or other read-only states). This prevents silent and opaque failures when the agent lacks full tool execution capabilities (specifically, shell execution).

What changed

Area Before After Reviewer focus
Go skill exporter Generated templates without explicit notices regarding execution environment. Appends executionModeDX notice to all generated routers, rules, and markdown operations. labs/12-product-engineering-loop/product-engineering-loop/export.go
Export Unit Tests No assertions checking for execution mode notices. Assertions added to verify that Execution Mode Notice: is generated across all adapters. labs/12-product-engineering-loop/product-engineering-loop/export_test.go
Release Notes No release note regarding this fix. Added an append-only release note fragment detailing the generic notice. labs/12-product-engineering-loop/boatstack-distribution/release-notes/2026-07-20-generic-execution-mode-dx.md

Review order

  1. Start with export.go to review the newly added executionModeDX constant and template string appends.
  2. Review the unit test assertions added to export_test.go.
  3. Check the added release note fragment.

Evidence

  • Go unit tests: Passed successfully with 100% success rate (ok github.com/operatorstack/boatstack/boatstack 24.180s).
  • Python integration tests: Successfully verified that generated frontmatter formats parse perfectly (Ran 18 tests in 7.676s OK).

Operational safety

Entirely non-destructive change. Adds documentation/instructions to the text of the exported skill files.

Known gaps and risks

None.

Rollout and rollback

  • Rollout: Standard. Merging to main compiles and projects the updated adapter skills to the public Boatstack distribution on next release.
  • Observability: Verification in agent-hosted chats.
  • Smallest safe rollback: Revert this commit.

…plates

Append a vendor-agnostic execution-mode warning to the end of all generated routers, rule files, and operation markdown commands. This notice instructs agents (Gemini, Claude, Cursor, Codex) to immediately halt and prompt the user to exit Plan Mode or Read-Only Mode before attempting to run Boatstack commands.

- export.go: Define executionModeDX constant and append it to commandBody, adapterSkill, and Cursor's rules.

- export_test.go: Assert that the Execution Mode Notice is generated correctly in routers, mdc rules, and markdown operations across all adapters.

- boatstack-distribution/release-notes: Add a release note fragment for the generic execution-mode notice.
@bigboateng
bigboateng merged commit b8bde5f into main Jul 20, 2026
4 checks passed
@bigboateng
bigboateng deleted the fix/generic-execution-mode-dx branch July 20, 2026 10:50
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