Skip to content

refactor(move): move the batch runner and batch policy into @agent-device/command-registry - #2388

Merged
thymikee merged 2 commits into
mainfrom
refactor/core-batch-command-registry
Sep 8, 2026
Merged

refactor(move): move the batch runner and batch policy into @agent-device/command-registry#2388
thymikee merged 2 commits into
mainfrom
refactor/core-batch-command-registry

Conversation

@thymikee

@thymikee thymikee commented Sep 7, 2026

Copy link
Copy Markdown
Member

Summary

Move 3 of the src/core -> owning-package sweep (#2335): src/core/batch.ts (the batch runner) and src/core/batch-policy.ts (the batch allowlist/normalization policy, which derives from the command-descriptor registry) move to @agent-device/command-registry behind new ./batch and ./batch-policy subpaths; batch.test.ts moves with them. The moved files keep their content (git rename detection; batch-policy only swaps its two self-package imports for relative ones, per package convention). The thirteen consumer import sites (src/daemon, src/commands, src/mcp, src/cli, src/client, src/sdk) now use the package subpaths, and command-registry declares its new @agent-device/kernel dependency. The public agent-device/batch entry (src/sdk/batch.ts) is unchanged in shape. 20 files, 47 insertions.

Validation

  • pnpm check:affected --run at 1e26e7bf53: all runnable checks passed — format, lint, typecheck, layering (R11 now 240 exported subpaths), di-seams, fallow, mcp-metadata, build, package (published-tarball closure + clean-install import of every entry point), integration-node, macos-coverage, vitest-related, integration-progress, replay-compat, daemon-wire-compat, plus the fail-open full set.
  • git diff -M90% --stat origin/main...HEAD: batch.ts and batch.test.ts are pure renames; batch-policy.ts differs only in its two now-relative self imports.
  • tsdown chunk groups sdk-batch / sdk-batch-runner and the fallow health baseline are re-pointed at the new location (gate commit).
  • No unresolved risks; rename-only, no behavior change.

@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown

Size Report

Metric Base Current Diff
Installed (including dependencies) 4.51 MB 4.51 MB -133 B
Package (unpacked) 4.51 MB 4.51 MB -133 B
Package (download) 1.34 MB 1.34 MB +89 B

Startup median (7 runs, lower is better):

Scenario Base Current Diff
CLI --version 25.6 ms 25.6 ms +0.1 ms
CLI --help 75.2 ms 71.4 ms -3.8 ms

@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-09-08 05:48 UTC

@thymikee

thymikee commented Sep 7, 2026

Copy link
Copy Markdown
Member Author

Reviewed 1e26e7b. The batch runner and policy move without changing behavior or the public batch entry. Consumers, package exports, test discovery, bundle groups, and the baseline follow the move. No findings; all checks pass. Ready for human review.

@thymikee thymikee added the ready-for-human Valid work that needs human implementation, judgment, or maintainer merge label Sep 7, 2026
@thymikee
thymikee merged commit 527a56a into main Sep 8, 2026
20 checks passed
@thymikee
thymikee deleted the refactor/core-batch-command-registry branch September 8, 2026 05:48
thymikee added a commit that referenced this pull request Sep 8, 2026
…cope-contracts

* origin/main: (25 commits)
  fix(ios): avoid duplicate alert routing queries (#2398)
  docs(adr): record the #2278 coupling audit and re-trace entry-to-platform hops (#2355)
  chore(gates): classify #2278 daemon-platform-runtime edges and ratchet handler session authority (#2354)
  test(capture): shrink the 3x crop fixture to avoid coverage timeouts (#2399)
  perf(ios): settle the first interaction on a deadline, not a fixed sleep (#2395)
  feat: support standalone Maestro clearState command (#2366)
  refactor(capture-kit): complete ADR 0019 end state — relocate snapshot and recording zones (#2385)
  refactor(ios): delete the unused snapshot plan interface (#2392)
  fix(conformance): isolate post-tap settling from app launch (#2390)
  test(remote): assert the proxied snapshot's whole wire conversation (#2387)
  fix(daemon): stop an attested tenant from downgrading its own isolation (#2386)
  refactor(ios): prune converged snapshot paths (#2383)
  refactor(move): move the batch runner and batch policy into @agent-device/command-registry (#2388)
  fix(remote): let a plain-session client read its own failure record (#2382)
  refactor(move): move replay divergence vocabulary into @agent-device/ad-replay (#2384)
  refactor(cli): take the CLI's vocabulary off runtime barrels (#2379)
  perf(cli): keep scroll vocabulary off gesture runtime (#2378)
  chore: drop the stale planned-operations closure approval (#2373)
  fix(gates): stop an eager-closure approval from turning main red on merge (#2375)
  fix(ios): honor the startup budget through a cold Simulator boot (#2325)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-human Valid work that needs human implementation, judgment, or maintainer merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant