Repository navigation
docs(openspec): propose feedback-channels - #477
Conversation
|
@claude /review |
|
Claude finished @theCodeDrift's task in 3m 28s —— View job Review completeReview mode: incremental — read 0 prior review thread(s) before reviewing.
This is a docs/OpenSpec-only PR (no package code touched), so no typecheck/lint/test run was needed or attempted. What I checked specifically, given CLAUDE.md's warning that a
[New] One finding, posted inline on Nothing else stood out — the proposal, design, and spec delta are unusually careful about exactly the OpenSpec archive/rename hazards this repo has been burned by before. |
2c81c8a to
7230c0c
Compare
… schema layer lands it
The one finding (tasks 1.1–1.3 pre-checked) is fixed in cb65006; those boxes are now left for the schema layer to check off. — AI Coding Agent |
Stack (root → tip):
Proposes splitting the agent feedback path into three channels, each sending to its own PostHog survey:
agent rule-feedback: the invited rule-authoring survey (01a0c7b9…). It is renamed fromagent feedback, and its questions, invite, cadence anddismissare unchanged.agent feedback: general feedback (01a11da4…), listed in the agent index.agent bug-report: bug reports (01a11da7…), listed in the agent index. No GitHub account is needed.feedback send --fromtakes a payload discriminated by a requiredkind(rule|general|bug) and validates it with zod. With telemetry off, nothing is sent and the CLI points the user to https://github.com/taskless/cli/issues.This PR carries the stack's single changeset (
patch, since the package is pre-1.0).This stack lands with
gh stack merge, which merges every PR tomainin one all-or-nothing operation, so no layer reachesmainon its own. That matters because the layers only work together. For example, #478 requireskindon the payload while the recipe the invite names still describes a single survey, and #480 listsbug-reportin the index before the skill routes to it.