Skip to content

build: remove monorepo and lerna #3508

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
eablack wants to merge 8 commits into
base: v11.0.0
Choose a base branch
from
Draft

build: remove monorepo and lerna #3508

eablack wants to merge 8 commits into from
+11,868 −18,406

Conversation

@eablack
Copy link
Contributor

@eablack eablack commented Feb 2, 2026

WIP

@eablack
remove monorepo structure. move everything from packages/cli to root.
8fab2d9
@eablack
switch from /lib to /dist, fix eslint config
1fad4d5
@eablack
remove unneeded ignores
f2fb425
@eablack
fix: regenerate package-lock.json after monorepo removal
462ea60 
The package-lock.json was out of sync with package.json after removing
the monorepo structure, causing npm ci to fail in CI. Regenerated the
lock file to resolve dependency mismatches.
@eablack eablack temporarily deployed to AcceptanceTests February 2, 2026 17:52 — with GitHub Actions Inactive
@eablack eablack temporarily deployed to AcceptanceTests February 2, 2026 17:52 — with GitHub Actions Inactive
@eablack
fix: update paths after monorepo removal
8fa0664 
Update test and config file paths that were still referencing the old
monorepo structure (packages/cli).
@eablack eablack temporarily deployed to AcceptanceTests February 2, 2026 17:59 — with GitHub Actions Inactive
@eablack eablack temporarily deployed to AcceptanceTests February 2, 2026 17:59 — with GitHub Actions Inactive
@eablack
fix: update oclif config to use dist directory and set test root
e659a0e 
After switching from /lib to /dist in commit 1fad4d5, the
oclif.config.mjs wasn't updated to point to the new directory.
This caused commands to not be discovered.

Also set OCLIF_TEST_ROOT environment variable in test init to help
@oclif/test find the CLI root after monorepo removal.
@eablack eablack temporarily deployed to AcceptanceTests February 2, 2026 18:40 — with GitHub Actions Inactive
@eablack eablack temporarily deployed to AcceptanceTests February 2, 2026 18:40 — with GitHub Actions Inactive
@eablack eablack temporarily deployed to AcceptanceTests February 2, 2026 18:40 — with GitHub Actions Inactive
@eablack eablack temporarily deployed to AcceptanceTests February 2, 2026 18:40 — with GitHub Actions Inactive
@eablack
fix: clean up git remotes created by apps:create tests
228badf 
The apps:create command creates actual git remotes when running in a
git repository. In tests, this was creating a 'heroku' remote pointing
to 'foobar' which then interfered with other tests that check for app
names from git remotes.

Added cleanup in afterEach to remove any heroku remotes created during
the tests.
@eablack
fix(test): add Procfile fixture for local command tests
c3e0a08 
After removing the monorepo structure, the example Procfile that local
command tests depended on was deleted. Created a test fixture at
test/fixtures/local/Procfile with the same content.

Updated tests that verify default Procfile loading to change directory
to the fixtures location before running, ensuring they can find the
default Procfile without explicitly specifying the path.
@eablack eablack temporarily deployed to AcceptanceTests February 3, 2026 00:44 — with GitHub Actions Inactive
@eablack eablack deployed to AcceptanceTests February 3, 2026 00:44 — with GitHub Actions Active
@eablack eablack temporarily deployed to AcceptanceTests February 3, 2026 00:44 — with GitHub Actions Inactive
@eablack eablack temporarily deployed to AcceptanceTests February 3, 2026 00:44 — with GitHub Actions Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@eablack