Skip to content

refactor: extract shared AI runner - #99

Merged
aksOps merged 1 commit into
mainfrom
codex/issue-81-ai-runner
Aug 17, 2026
Merged

refactor: extract shared AI runner#99
aksOps merged 1 commit into
mainfrom
codex/issue-81-ai-runner

Conversation

@aksOps

@aksOps aksOps commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • move the skill runner, board tools, built-in skills, guarded link client, and tool-calling probe into internal/ai
  • expose a direct-store runner and tool constructors for local TUI and CLI callers
  • leave the server as a thin adapter that preserves the existing HTTP request and response contracts
  • move the tool and skill tests with the implementation and add direct runner, probe, and guard coverage

Why

The server owned AI infrastructure that did not depend on HTTP. Local frontends would otherwise need to construct a server merely to run a skill against *store.Store, which is architecture by hostage situation.

Validation

  • env GOFLAGS=-buildvcs=false sh scripts/check-go-coverage.sh (96.8% total; internal/ai 95.5%; internal/server 98.2%)
  • env GOFLAGS=-buildvcs=false go vet ./...
  • env GOFLAGS=-buildvcs=false go test -race ./... -count=1
  • npm test (60 files, 695 tests)
  • npm run build
  • sh scripts/check-go-format.sh
  • git diff --check

Closes #81

@sonarqubecloud

Copy link
Copy Markdown

@aksOps
aksOps marked this pull request as ready for review August 17, 2026 13:07
@aksOps
aksOps merged commit ce606d6 into main Aug 17, 2026
9 checks passed
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.

Extract the AI runner from the server into a shared package

1 participant