Skip to content

Commit 50ee986

Browse files
committed
refactor!: remove dead packages with no importers
Delete 30 packages that nothing imports, are not referenced by CI or scripts, and are not documented as active seams: deps, engine/async, engine/code, engine/prompt, engine/trajectory, fastcopy, feature/ide, feature/voice, hunktracker, localize, magicdocs, observability/{alerts,conformance,logfield}, preset, profile, system/{bus,retention,shutdown}, task, tools, container, bench, tui/diff, filestate, gitsnapshot, lockutil, intelligence/harness, system/cron, and the unused root context/resilience/system files. Kept: context/repomap (used by cmd/options.go), resilience/{health,retry, ratelimit} (used by engine + verify-milestone.sh), and testaudit (CI package-boundary guard). Drop the README adopted-capabilities rows and doc references for the removed packages.
1 parent 9286d94 commit 50ee986

101 files changed

Lines changed: 5 additions & 19915 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

‎README.md‎

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -269,9 +269,6 @@ Features adopted from open-source agent projects. All are off by default unless
269269
| App verification | `AppVerify` tool | Boot-smoke check with readiness polling and evidence artifacts |
270270
| Media generation | `GenerateMedia` tool | Image/video generation with local persistence. Backend via `tool.SetMediaEngine`; an OpenAI-compatible client ships in `eyrie/client` (`ImageClient`), wired by the host (boundary-guarded — hawk routes through the eyrie facade) |
271271
| Voice transcription | Telegram voice notes + `stt` package | Transcribe Telegram voice/audio into the prompt. Backend via `stt.SetTranscriber`; an OpenAI-compatible client ships in `eyrie/client` (`AudioClient`), wired by the host |
272-
| Git-tree file snapshots | `internal/gitsnapshot` | Content-addressed tree capture/diff/preview/restore |
273-
| Turn-boundary rewind | `internal/filestate` | Per-prompt before/after snapshots with durable store |
274-
| Continual harness | `internal/intelligence/harness` | Versioned, evidence-backed refinement of supplemental prompts/memories/skills/subagents with rollback |
275272
| Bounded autonomous budgets | `internal/engine` (`AutonomousBudget`) | Track turns/tokens/time/continuations; report why a run stopped (budget vs gate-passed vs error) |
276273
| Agent family messaging | `internal/multiagent` (`FamilyMessenger`) | Direct parent/sibling/child messages with pending caps + rate limits |
277274
| Path reservations | `internal/multiagent` ledger | Detect overlapping-file changes between parallel branches |
@@ -281,7 +278,6 @@ Features adopted from open-source agent projects. All are off by default unless
281278
| Token-cheaper file views | `Read` tool `--minify` | Read-only, comment-stripped, whitespace-dense file view (Go via `go/parser`; other languages string-aware; never touches disk) — fewer tokens per read |
282279
| Classified provider hints | `internal/errhint` | Buckets provider errors (Auth/RateLimit/Connectivity/ModelNotFound/ContextOverflow) into a one-line fixable next step. Wired into TUI error rows (`friendlyErrorMessage`) and `hawk exec` CLI errors |
283280
| Atomic install transactions | `internal/installtxn` | Cross-process staged install/remove with rollback. Wired into skill install (atomic `SKILL.md` publish) |
284-
| Stale-lock reclaim | `internal/lockutil` | Race-correct atomic reclaim of O_EXCL lock files with live-restore (ready for O_EXCL lock sites) |
285281
| Test command discovery | `internal/testrunner` | Auto-detect test/verify commands (Go/npm/bun/pnpm/yarn/pytest/cargo) and parse runner output into structured results. Wired into `hawk verify` |
286282
| Circuit breaker | `internal/circuitbreaker` | Closed/open/half-open retry-storm protection with cooldown. Wired into auto-compaction (cooldown + half-open auto-retry) |
287283
| Smart turn routing | `internal/smartrouting` | Deterministic simple/strong turn classifier with fail-toward-strong safety. Wired into per-turn model selection (`settings.smart_routing`) |

‎docs/COMPETITIVE.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ Verified from source in this repo:
1616
- Custom providers: supported — `internal/config/settings.go:50` (`custom_providers`), `internal/config/eyrie_engine.go:32-50`.
1717
- Unwired backends: `internal/tool/computer_use.go:67-94` (`SetComputerBackend`, nil default), `internal/tool/media_generation.go:69-71` (`SetMediaEngine`, nil default).
1818
- Terminal detect covers kitty/ghostty/wezterm/alacritty names (`internal/ui/icons/detect_test.go:56`); Kitty graphics protocol not implemented (see `docs/plans/pi-adoption-plan.md:25`).
19-
- Bench infra exists (`internal/bench/suite.go`, `internal/feature/eval/`, `make bench`) but README publishes no numbers.
19+
- Bench infra exists (`internal/feature/eval/`, `make bench`) but README publishes no numbers.
2020

2121
External star counts below are approximate web-search snapshots (2026-09-08), not repo-verified. Treat as order-of-magnitude traction, not exact rankings. hawk is pre-release (`VERSION`: `0.0.1`, `README.md:40-44` source-build primary) — it competes on architecture, not stars.
2222

‎docs/design/ECOSYSTEM-MARKETPLACE.md‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -231,7 +231,7 @@ Concrete reuse map — what is *already there* and what each piece becomes:
231231
| Hook bundling | `ManifestHook{Event,Command,Priority}` | `manifest_v2.go:32` |
232232
| MCP server bundling | MCP client/loader | `hawk/internal/mcp/mcp.go`, `hawk/internal/tool/mcp_tool.go` |
233233
| Sub-agent bundling | Persona system (YAML frontmatter MD) | `hawk/internal/multiagent/agents/` (cf. `TOP20_COMPARISON.md:62`) |
234-
| Rule/skill discovery precedence | `DefaultRuleSources`, `RuleDiscoverer` | `hawk/internal/context/rules.go:20`, `:46` |
234+
| Rule/skill discovery precedence | `DefaultRuleSources`, `RuleDiscoverer` | `hawk/internal/plugin/` rule discovery |
235235
| Trust — static scan | `malware_check.go` (blocked/suspicious regex) | `hawk/internal/plugin/malware_check.go:19` |
236236
| Trust — unicode/homoglyph | `audit.go` (`AuditFinding`, severities) | `hawk/internal/plugin/audit.go:12-22` |
237237
| Registry generators/validators | Python tooling | `starling/tools/{update_registry,registry_schema,validate_skill,package_skill,sync_marketplace}.py` |
@@ -366,7 +366,7 @@ The estimate is bounded on the low side because the hardest plumbing — the reg
366366
- `hawk/internal/plugin/malware_check.go:19`
367367
- `hawk/internal/plugin/audit.go:12`
368368
- `hawk/internal/tool/skill.go:62,91`
369-
- `hawk/internal/context/rules.go:20,46,97`
369+
- `hawk/internal/plugin/` rule discovery
370370
- `hawk/cmd/skills_cmd.go:16-211`
371371
- `hawk/internal/mcp/mcp.go`, `hawk/internal/tool/mcp_tool.go`
372372
- `TOP20_COMPARISON.md:65,74,83,181,227,232,240,241`

‎docs/design/HAWK-CLOUD-SAAS.md‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -315,7 +315,7 @@ This is the crux: **what is reusable today vs. net-new.**
315315
| Sandbox executor interface | `internal/sandbox/container.go:18-21` (`containerExecutor`: `Exec`, `Running`) | `CloudSandbox` implements the same interface → drop-in. Callers don't know if they're on local Docker or a cloud microVM. |
316316
| Sandbox lifecycle manager | `internal/sandbox/snapshot_sandbox.go:52-228` (`Create/Pause/Resume/Snapshot/Restore/List/Cleanup`) | Existing pause/resume/snapshot semantics map cleanly onto E2B/Daytona pause+snapshot APIs; the manager abstraction guides the `CloudSandbox` API shape. |
317317
| Messaging gateways | `internal/daemon/gateway.go`, `telegram.go`, `discord.go`, `slack.go` | Already forward to `/v1/chat` via `forwardToHawk` (`gateway.go:17`) with bearer auth. In cloud they forward to the tenant-scoped chat endpoint with the org's key — minimal change. |
318-
| Cron engine | `internal/system/cron/cron.go` | Foundation for cloud scheduled runs (Routines), per `TOP20_COMPARISON.md:48`. Out of scope here but shares the worker plane. |
318+
| Cron engine | (not yet implemented) | Cloud scheduled runs (Routines) are planned, per `TOP20_COMPARISON.md:48`. Out of scope here but shares the worker plane. |
319319

320320
### Net-new (must build)
321321

@@ -387,7 +387,7 @@ credits hit zero.
387387
- **M2.3** JetBrains extension (ACP).
388388
- **M2.4** Org-scoped provider config (BYO keys / BYO model endpoints), aligning with eyrie multi-tenant proxy (`TOP20_COMPARISON.md:96`).
389389
- **M2.5** Audit logging, IT-managed policy tiers, self-hosted/on-prem distribution.
390-
- **M2.6** Cloud Routines (scheduled/triggered runs) on the worker plane via `internal/system/cron/cron.go` (`TOP20_COMPARISON.md:48`).
390+
- **M2.6** Cloud Routines (scheduled/triggered runs) on the worker plane (`TOP20_COMPARISON.md:48`).
391391

392392
---
393393

‎internal/bench/bench_test.go‎

Lines changed: 0 additions & 114 deletions
This file was deleted.

‎internal/bench/suite.go‎

Lines changed: 0 additions & 122 deletions
This file was deleted.

0 commit comments

Comments
 (0)