Skip to content

Liberate: add /migrate allias for /liberate#4111

Open
nightnei wants to merge 1 commit into
trunkfrom
stu-1973-add-migrate-alias-for-liberate
Open

Liberate: add /migrate allias for /liberate#4111
nightnei wants to merge 1 commit into
trunkfrom
stu-1973-add-migrate-alias-for-liberate

Conversation

@nightnei

@nightnei nightnei commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Related issues

How AI was used in this PR

AI assisted

Proposed Changes

In the data-liberation-agent, there's an alias to the /liberate skill called /migrate that Matthew added because the word liberate is probably not something that folks would easily remember.

With this PR, we are adding the alias as the first step, and later we can see how it goes.

Testing Instructions

  1. npm run cli:build
  2. node apps/cli/dist/cli/main.mjs code
  3. Type "/mig" and assert that you see the command /migrate with mentioning that it's an allias for /liberate

@nightnei nightnei requested a review from sejas July 7, 2026 13:24
@nightnei nightnei self-assigned this Jul 7, 2026

@sejas sejas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was able to invoke the Liberate skill using /migrate command.

Image

Another alternative we could consider is keeping the /liberate command and just allowing to search by the description where we could mention migrate instead of import.

Image

@wpmobilebot

Copy link
Copy Markdown
Collaborator

📊 Performance Test Results

Comparing a35671b vs trunk

app-size

Metric trunk a35671b Diff Change
App Size (Mac) 1411.96 MB 1411.97 MB +0.00 MB ⚪ 0.0%

site-editor

Metric trunk a35671b Diff Change
load 1052 ms 1056 ms +4 ms ⚪ 0.0%

site-startup

Metric trunk a35671b Diff Change
siteCreation 6518 ms 6510 ms 8 ms ⚪ 0.0%
siteStartup 2385 ms 2392 ms +7 ms ⚪ 0.0%

Results are median values from multiple test runs.

Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change (<50ms diff)

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.

3 participants