Skip to content

refactor(packages)!: delete code-review; fold agent-events, artifact-ui, tool-registry-publish into their consumers (CL-8390) - #845

Merged
TheGreatAxios merged 3 commits into
mainfrom
cl-8390-fold-packages
Sep 18, 2026
Merged

TheGreatAxios merged 3 commits into
mainfrom
cl-8390-fold-packages

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Contributor

Summary

  • Deleted packages/code-review (dead: no app/workflow/deploy path imported it; only a test-fixture string mentioned the name).
  • Folded packages/agent-events into packages/agent-directory/src/agent-events.ts, keeping only the two readers one-shot-prompt.ts actually imports (connectorReplyContent, messageRunEnded); dropped the unused readers (inferenceDoneBlocks, toolDoneResult, messageRunBracket, messageRunStarted) and their tests.
  • Moved packages/artifact-ui (the ArtifactSummary type and the Files kind-nav mapping) into apps/web/src/library/. The hub no longer depends on the package: it now buckets @corbits/artifacts' own ArtifactListRow through a small hub-local apps/hub/src/library-kind-segments.ts kept in step by hand with the web app's copy.
  • Deleted packages/tool-registry-publish. Its bundling-based publish loop (bun-build + tar) is gone; a new apps/web/src/tools/registry-publish.ts reuses myra-deploy.ts's own packTarball to pack each tools/* package's raw source (directories read live off the workspace via import.meta.glob, so a new tools/* package needs no directory-list edit) and publish it the same way Myra's own tarball is published. tarballCoversPackage and the corbits-tools registry name moved into a new packages/agent-directory/src/tool-registry.ts; tools/access's dependency on the deleted package was unused and dropped.

Notes

  • No new call site wires publishToolPackageRegistry in yet (nothing did before either — the old package's publish loop had no caller in this repo), so this is a like-for-like move with no behavior change.
  • bun run check and apps/web's bun run build are green on this branch after rebasing onto origin/main (which independently deleted tools/{linear,github,granola,reddit,web-search,gotenberg} and code-review in the interim).

Test plan

  • bun run check (typecheck, lint, fmt:check, test) — green, 1770 tests passing
  • apps/web: bun run build — succeeds
  • Repo-wide grep confirms no remaining references to any of the four deleted packages outside historical comments

@linear-code

linear-code Bot commented Sep 18, 2026

Copy link
Copy Markdown

CL-8390

@TheGreatAxios
TheGreatAxios merged commit 6d6683c into main Sep 18, 2026
4 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.

1 participant