Ver código fonte

Document hardware handoff

Paul Klumpp 2 semanas atrás
pai
commit
7acf462346
2 arquivos alterados com 49 adições e 1 exclusões
  1. 35 1
      PLAN.md
  2. 14 0
      docs/superpowers/plans/2026-09-25-impact80-effects.md

+ 35 - 1
PLAN.md

@@ -10,7 +10,7 @@
 - ✅ Per-zone `rgb info` JSON implemented
 - ✅ All RGB subcommands implemented (effect, brightness, speed, color, mode, enable, disable, info)
 - ✅ Automated tests, vet, formatting, and build verification
-- ⚠️ Hardware smoke discovery succeeds, but the connected Impact 80 is blocked by root-only `/dev/hidraw7` permissions
+- ⚠️ Automated implementation is complete; hardware smoke acceptance is pending. The last observed raw device was `/dev/hidraw7` (`root:root`, mode `600`); it later disappeared, so use the currently discovered path.
 
 ## Zone-Aware Lighting Semantics
 
@@ -108,3 +108,37 @@ have permission to access its hidraw device. In the acceptance environment,
 `/dev/hidraw7` is `root:root` with mode `600`, so discovery succeeds but all
 HID opens fail with `permission denied`. No persistent profile storage or
 EEPROM save command is included.
+
+## Handoff Status
+
+Implementation is complete on `feature/impact80-effects` at commit `62852e7`.
+All task reviews and the final whole-branch code review passed. Automated
+checks pass:
+
+```bash
+gofmt -w .
+go test -count=1 ./...
+go vet ./...
+go build -o /tmp/wobkey ./cmd/wobkey/
+git diff --check
+```
+
+The only open acceptance item is the live hardware smoke sequence. At the last
+attempt, discovery found the Impact 80 but HID access was denied; the device
+then disappeared. Do not assume `/dev/hidraw7`; run `keyboard info` after
+connecting the keyboard and use the path it reports.
+
+### Continue With Hardware
+
+1. Check out `feature/impact80-effects` and build `/tmp/wobkey`.
+2. Connect the Impact 80 and run `/tmp/wobkey keyboard info`.
+3. Verify the current hidraw path and grant access using the system’s existing
+   device-permission policy. If temporary access is explicitly chosen, record
+   the original mode and restore it after testing.
+4. Run the smoke commands in the Verification section above.
+5. Confirm the default unsupported-zone warning, explicit Backlight effect,
+   three-zone `rgb info` JSON, and final enabled state.
+6. Record the hardware result here, rerun the automated checks, and only then
+   merge the feature branch into `main`.
+
+Do not stage `graphify-out/` unless explicitly requested.

+ 14 - 0
docs/superpowers/plans/2026-09-25-impact80-effects.md

@@ -329,3 +329,17 @@ git commit -m "Document zone-aware RGB commands"
 ```
 
 Do not stage `graphify-out/` unless the user explicitly requests generated graph artifacts.
+
+## Execution Handoff — 2026-09-25
+
+- Tasks 1–5 and the documentation/automated-verification portions of Task 6
+  are implemented in `feature/impact80-effects` through commit `62852e7`.
+- All task reviews and the final whole-branch code review are approved.
+- `go test -count=1 ./...`, `go vet ./...`, `go build`, `gofmt`, and
+  `git diff --check` pass.
+- Task 6’s live hardware acceptance remains open. The last run discovered the
+  Impact 80 at `/dev/hidraw7`, but the device was root-owned mode `600`; it
+  later disappeared, so the next operator must discover the current path.
+- Continue with the hardware procedure in `PLAN.md`, record the result, and
+  merge the feature branch into `main` only after the three-zone JSON and final
+  enabled state have been verified.