Skip to content

Clarify /start skill gate messages with intent classification#98

Merged
sebastientaggart merged 2 commits into
devfrom
feature/clarify-start-gates
Apr 11, 2026
Merged

Clarify /start skill gate messages with intent classification#98
sebastientaggart merged 2 commits into
devfrom
feature/clarify-start-gates

Conversation

@sebastientaggart

Copy link
Copy Markdown
Member

Both human gate messages in the /start skill now suggest "go" as a fast-path shorthand and include agent-side intent classification rules — affirmative (proceed), affirmative with conditions (discuss first), questions (discuss), and abandonment (stop). This replaces the previous yes/no framing with clearer next-step options for Case A (approach confirmation) and Case B (resume confirmation).

Issue #96

@sebastientaggart

Copy link
Copy Markdown
Member Author

Review Summary

Verdict: APPROVE

Findings

  • [WARNING] Nested double quotes in gate messages create ambiguous quoting — "Does this approach sound right? Type "go" to create..." has inner "go" breaking the outer "..." pair. Consider using single quotes or backticks for the inner token (e.g., Type 'go' or Type `go`) so the sentence boundary is unambiguous when an LLM parses it. This appears in both Case A (Step 2) and Case B (Step 2) gate messages across all 5 adapter files. Not blocking because LLMs will likely infer the intent correctly from context, but it is technically malformed quoting.

No other issues found. The intent classification rules are consistent across all adapter files, the changes match the PR description, and the sync hashes are updated. No secrets, no security concerns.

@sebastientaggart

Copy link
Copy Markdown
Member Author

Review Summary

Verdict: APPROVE

Findings

No issues found. The previous WARNING about nested double quotes has been resolved — go is now backtick-quoted consistently across all 10 gate messages (5 adapter files x 2 gates each). Intent classification blocks are identical across all adapters, the changes match the PR description, sync hashes are updated, and no secrets or security concerns are present.

@sebastientaggart
sebastientaggart merged commit d5ed672 into dev Apr 11, 2026
2 checks passed
@sebastientaggart
sebastientaggart deleted the feature/clarify-start-gates branch April 11, 2026 13:25
This was referenced Apr 11, 2026
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