fix(migration-legacy): import slugifyWorkDirName from agent-core

[Arvuno]

Related Issue

The TODO at the top of packages/migration-legacy/src/sessions/workdir-bucket.ts is resolved by this PR. No upstream issue needed — the duplicate was flagged in-source.

Problem

packages/migration-legacy/src/sessions/workdir-bucket.ts carried a local copy of slugifyWorkDirName identical to the canonical implementation in packages/agent-core/src/utils/workdir-slug.ts. The local copy was marked with a TODO to delete it once the function became public on @moonshot-ai/agent-core.

This is a small class of bug: if the canonical implementation ever changes (length cap, replacement regex, edge-case handling), the migrator would silently produce a different bucket name than the running app, making migrated sessions invisible in the session picker.

What changed

• Export slugifyWorkDirName from packages/agent-core/src/index.ts so in-monorepo consumers can import it directly.
• Replace the local duplicate in packages/migration-legacy/src/sessions/workdir-bucket.ts with import { slugifyWorkDirName } from '@moonshot-ai/agent-core'.
• Remove the TODO comment block.
• Add a .changeset/ entry noting the internal dependency update.

No user-visible behavior change. The migrator's computeWorkdirBucket produces the same output as before for the same inputs — both call sites now share the canonical implementation, so they cannot drift.

Why this fits the repo

This is exactly the case the project's monorepo layout was set up for: a canonical utility in agent-core consumed by a sibling package. The TODO explicitly asked for this refactor once the export became available; this PR makes the export available and follows through.

Checklist

• I have read the CONTRIBUTING document.
• I have linked a related issue, or explained the problem above. (In-source TODO, not an issue.)
• I have added tests that prove my feature works. (No new tests needed — the existing test at packages/migration-legacy/test/sessions/workdir-bucket.test.ts continues to validate computeWorkdirBucket output and the function is unchanged at the call site.)
• Ran gen-changesets skill, or this PR needs no changeset. (Changeset added under .changeset/fix-migration-legacy-shared-slugify.md.)
• Ran gen-docs skill, or this PR needs no doc update. (Internal refactor, no user-facing docs change.)

Environment note

I was unable to run pnpm install / pnpm test locally because the repository requires Node >=24.15.0 and the available toolchain in this environment is Node 22.22.2 (engine-strict is on, so install fails fast). The diff is small and self-contained:

• One new export line in packages/agent-core/src/index.ts.
• Net -17 lines in workdir-bucket.ts (duplicate function removed, one import added, comment refreshed).
• The exported function's body is byte-identical to the local copy that was removed, so behavior is preserved.

I would appreciate a CI run from a maintainer to confirm the test suite still passes.

[changeset-bot]

🦋 Changeset detected

Latest commit: 183242b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages

Name	Type
@moonshot-ai/agent-core	Patch
@moonshot-ai/kimi-code	Patch
@moonshot-ai/migration-legacy	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 767d1006be

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

[chatgpt-codex-connector]

Include the changed migrator package in the changeset

This changeset omits @moonshot-ai/migration-legacy even though the commit changes packages/migration-legacy/src/sessions/workdir-bucket.ts, so changeset version will not bump that package or update its changelog for the migrator output change. The repo's changeset rules require listing packages whose source/output changed, with @moonshot-ai/kimi-code added separately when the internal change enters the CLI bundle; please add the migrator package to the frontmatter.

Useful? React with 👍 / 👎.