Skip to content

feat(wire): integrate errhint, testrunner, installtxn into hawk - #251

Merged
Patel230 merged 1 commit into
mainfrom
feat/zero-wiring
Aug 24, 2026
Merged

feat(wire): integrate errhint, testrunner, installtxn into hawk#251
Patel230 merged 1 commit into
mainfrom
feat/zero-wiring

Conversation

@Patel230

@Patel230 Patel230 commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Summary

Wires the Zero-adopted mechanisms (from PR #250) into hawk's running system:

errhint -> error surfacing

  • cmd/error_classify.go: friendlyErrorMessage now appends errhint.TUIHint(err) — every TUI error row gets a one-line fixable next step for provider errors. Conservative: local errors draw no hint.
  • cmd/exec.go: the headless error event now prints errhint.CLIHint for text output (slash commands don't apply headless).

testrunner -> verify loop

  • cmd/verify_cmd.go: hawk verify now discovers project test/verify commands via testrunner.Detect, runs them, and prints structured summaries (passed/total + failure file/message) via testrunner.ParseSummary. Purely additive — existing security-log and governance checks untouched.

installtxn -> skill install

  • internal/plugin/registry.go: skill SKILL.md manifest now published via installtxn.WriteFileAtomically (atomic rename, no partial write).

lockutil: no clean fit — hawk's lock sites use kernel flock (session autosave), not O_EXCL files, so it stays a ready primitive rather than a forced fit.

Verification

  • New tests: TestFriendlyErrorMessageAppendsProviderHint / NoHintForLocalError, TestRunWorkspaceChecksDetectsGoProject / EmptyDir
  • go build ./..., go vet, full cmd/plugin/errhint/testrunner/installtxn/lockutil/minify/tool suites green
  • Boundary guard + govulncheck + full pre-push test suite green

@Patel230
Patel230 merged commit b46fb32 into main Aug 24, 2026
26 checks passed
@Patel230
Patel230 deleted the feat/zero-wiring branch August 24, 2026 06: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.

1 participant