Oclif4 by pviti · Pull Request #116 · commercelayer/commercelayer-cli-plugin-imports

pviti · 2026-03-27T08:47:59Z

No description provided.

BREAKING CHANGE: upgrade to oclif core v4 and add biome

# [5.0.0-oclif4.1](v4.3.1...v5.0.0-oclif4.1) (2026-03-26) ### Features * upgrade to oclif core v4 and add biome ([c430b37](c430b37)) ### BREAKING CHANGES * upgrade to oclif core v4 and add biome

BREAKING CHANGE: require node to v22

…er-cli-plugin-imports into oclif4

# [5.0.0-oclif4.2](v5.0.0-oclif4.1...v5.0.0-oclif4.2) (2026-03-26) ### Features * require node to v22 ([cc1599d](cc1599d)) ### BREAKING CHANGES * require node to v22

…er-cli-plugin-imports into oclif4

# [5.0.0-oclif4.3](v5.0.0-oclif4.2...v5.0.0-oclif4.3) (2026-03-26) ### Bug Fixes * fix tests ([0b06774](0b06774))

…er-cli-plugin-imports into oclif4

# [5.0.0-oclif4.4](v5.0.0-oclif4.3...v5.0.0-oclif4.4) (2026-03-26) ### Features * add import size flag ([4fa5433](4fa5433))

…er-cli-plugin-imports into oclif4

Copilot

Pull request overview

This PR updates the plugin to support the oclif v4 toolchain and associated ecosystem updates (dependencies, tests, and release automation).

Changes:

Upgraded oclif core to v4 and updated Commerce Layer CLI dependencies to the oclif4 dist-tag variants.
Migrated tests from the legacy @oclif/test “fancy-test” API to runCommand.
Updated release config and GitHub Actions workflows (semantic-release branches, action major versions, CodeQL).

Reviewed changes

Copilot reviewed 16 out of 17 changed files in this pull request and generated 16 comments.

Show a summary per file

File	Description
test/commands/imports/create.test.ts	Migrates to `runCommand`-based test harness
test/commands/imports/delete.test.ts	Migrates to `runCommand`-based test harness
test/commands/imports/details.test.ts	Migrates to `runCommand`-based test harness
test/commands/imports/group.test.ts	Migrates to `runCommand`-based test harness
test/commands/imports/index.test.ts	Migrates to `runCommand`-based test harness
test/commands/imports/list.test.ts	Migrates to `runCommand`-based test harness
test/commands/imports/types.test.ts	Migrates to `runCommand`-based test harness
src/commands/imports/noc.ts	Adds an oclif parse call in the NoC command
src/base.ts	Updates CommandError typing for oclif v4 (`Interfaces.CommandError`)
package.json	Bumps Node engine requirement and updates deps/devDeps for oclif v4; adjusts release script
pnpm-lock.yaml	Locks upgraded dependency graph for oclif v4 and related packages
README.md	Updates generated usage/docs formatting (e.g., optional `[ID]` for `imports`)
CHANGELOG.md	Adds prerelease entries for the new `oclif4` track
.releaserc	Adds `oclif4` prerelease branch config
.github/workflows/semantic-release.yml	Adds `oclif4` branch + bumps action versions
.github/workflows/publish.yml	Updates Slack GitHub Action major version + incoming webhook input
.github/workflows/codeql-analysis.yml	Bumps checkout + CodeQL action major versions

Files not reviewed (1)

pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-03-27T08:58:23Z


+describe('imports:create', () => {
+  it('runs NoC', async () => {
+    const { stdout } = await runCommand<{ name: string }>(['imports:noc'])


runCommand<T>()’s generic parameter types the command’s return value (result), not stdout. Since this test doesn’t use result, consider removing the generic (or changing it to string for imports:noc) to avoid misleading typing.

Suggested change

const { stdout } = await runCommand<{ name: string }>(['imports:noc'])

const { stdout } = await runCommand(['imports:noc'])

Copilot · 2026-03-27T08:58:23Z

-      expect(ctx.stdout).to.contain('-= NoC =-')
-    })

+describe('imports:create', () => {


These tests are named as if they validate imports:create, but they actually execute the hidden imports:noc command. This doesn’t exercise the imports:create command at all and can give a false sense of coverage. Consider running imports:create (with API calls stubbed/mocked) or renaming/consolidating these specs to reflect that they are only a smoke test for imports:noc.

Suggested change

describe('imports:create', () => {

describe('imports:noc', () => {

Copilot · 2026-03-27T08:58:23Z

-      expect(ctx.stdout).to.contain('-= NoC =-')
-    })

+describe('imports:details', () => {


These tests are named as if they validate imports:details, but they actually execute the hidden imports:noc command. This doesn’t exercise the imports:details command at all and can give a false sense of coverage. Consider running imports:details (with API calls stubbed/mocked) or renaming/consolidating these specs to reflect that they are only a smoke test for imports:noc.

Suggested change

describe('imports:details', () => {

describe('imports:noc', () => {

Copilot · 2026-03-27T08:58:24Z

+describe('imports:types', () => {
+  it('runs NoC', async () => {


This test suite is labeled imports:types, but it executes imports:noc. If the intent is to validate the imports:types output, it should run imports:types; otherwise, rename/consolidate this spec so it doesn’t imply coverage of that command.

Suggested change

describe('imports:types', () => {

it('runs NoC', async () => {

describe('imports:noc', () => {

it('runs imports:noc', async () => {

Copilot · 2026-03-27T08:58:25Z


+describe('imports:list', () => {
+  it('runs NoC', async () => {
+    const { stdout } = await runCommand<{ name: string }>(['imports:noc'])


runCommand<T>()’s generic parameter types the command’s return value (result), not stdout. Since this test doesn’t use result, consider removing the generic (or changing it to string for imports:noc) to avoid misleading typing.

Suggested change

const { stdout } = await runCommand<{ name: string }>(['imports:noc'])

const { stdout } = await runCommand(['imports:noc'])

Copilot · 2026-03-27T08:58:26Z


+describe('imports:types', () => {
+  it('runs NoC', async () => {
+    const { stdout } = await runCommand<{ name: string }>(['imports:noc'])


runCommand<T>()’s generic parameter types the command’s return value (result), not stdout. Since this test doesn’t use result, consider removing the generic (or changing it to string for imports:noc) to avoid misleading typing.

Suggested change

const { stdout } = await runCommand<{ name: string }>(['imports:noc'])

const { stdout } = await runCommand(['imports:noc'])

Copilot · 2026-03-27T08:58:26Z

-      expect(ctx.stdout).to.contain('-= NoC =-')
-    })

+describe('imports:group', () => {


These tests are named as if they validate imports:group, but they actually execute the hidden imports:noc command. This doesn’t exercise the imports:group command at all and can give a false sense of coverage. Consider running imports:group (with API calls stubbed/mocked) or renaming/consolidating these specs to reflect that they are only a smoke test for imports:noc.

Suggested change

describe('imports:group', () => {

describe('imports:noc', () => {

Copilot · 2026-03-27T08:58:26Z

+describe('imports:list', () => {
+  it('runs NoC', async () => {


This test suite is labeled imports:list, but it executes imports:noc. If the intent is to validate listing behavior, it should run imports:list and assert on its output (with API calls stubbed/mocked); otherwise, rename/consolidate the spec so it doesn’t imply coverage of imports:list.

Suggested change

describe('imports:list', () => {

it('runs NoC', async () => {

describe('imports:noc', () => {

it('runs imports:noc', async () => {

# [5.0.0-oclif4.5](v5.0.0-oclif4.4...v5.0.0-oclif4.5) (2026-03-27) ### Bug Fixes * update dependencies ([636fdfc](636fdfc))

pviti · 2026-03-27T09:39:43Z

🎉 This PR is included in version 5.0.0-oclif4.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

# [5.0.0-oclif4.6](v5.0.0-oclif4.5...v5.0.0-oclif4.6) (2026-04-27) ### Bug Fixes * fix biome config ([556738a](556738a)) * run tests in updates workflow ([547ce89](547ce89)) * update dependencies and add updates workflow ([59e61f1](59e61f1))

pviti · 2026-04-27T14:10:19Z

🎉 This PR is included in version 5.0.0-oclif4.6 🎉

The release is available on:

Your semantic-release bot 📦🚀

pviti and others added 15 commits March 24, 2026 10:47

feat: upgrade to oclif core v4 and add biome

c430b37

BREAKING CHANGE: upgrade to oclif core v4 and add biome

Merge branch 'main' into oclif4

5df5340

chore(release): 5.0.0-oclif4.1 [skip ci]

70095b8

# [5.0.0-oclif4.1](v4.3.1...v5.0.0-oclif4.1) (2026-03-26) ### Features * upgrade to oclif core v4 and add biome ([c430b37](c430b37)) ### BREAKING CHANGES * upgrade to oclif core v4 and add biome

feat: require node to v22

cc1599d

BREAKING CHANGE: require node to v22

Merge branch 'oclif4' of https://github.com/commercelayer/commercelay…

3155a91

…er-cli-plugin-imports into oclif4

chore(release): 5.0.0-oclif4.2 [skip ci]

546acbf

# [5.0.0-oclif4.2](v5.0.0-oclif4.1...v5.0.0-oclif4.2) (2026-03-26) ### Features * require node to v22 ([cc1599d](cc1599d)) ### BREAKING CHANGES * require node to v22

fix: fix tests

0b06774

Merge branch 'oclif4' of https://github.com/commercelayer/commercelay…

0bb4651

…er-cli-plugin-imports into oclif4

chore(release): 5.0.0-oclif4.3 [skip ci]

f0f47fd

# [5.0.0-oclif4.3](v5.0.0-oclif4.2...v5.0.0-oclif4.3) (2026-03-26) ### Bug Fixes * fix tests ([0b06774](0b06774))

Merge branch 'main' into oclif4

192c563

Merge branch 'oclif4' of https://github.com/commercelayer/commercelay…

515d0ba

…er-cli-plugin-imports into oclif4

chore(release): 5.0.0-oclif4.4 [skip ci]

e917273

# [5.0.0-oclif4.4](v5.0.0-oclif4.3...v5.0.0-oclif4.4) (2026-03-26) ### Features * add import size flag ([4fa5433](4fa5433))

chore: fix release script

df831f5

Merge branch 'oclif4' of https://github.com/commercelayer/commercelay…

3a4db77

…er-cli-plugin-imports into oclif4

chore: fix release script

0c3bc9c

pviti requested a review from Copilot March 27, 2026 08:48

Copilot started reviewing on behalf of pviti March 27, 2026 08:48 View session

Copilot AI reviewed Mar 27, 2026

View reviewed changes

pviti and others added 2 commits March 27, 2026 10:38

fix: update dependencies

636fdfc

chore(release): 5.0.0-oclif4.5 [skip ci]

113f4d6

# [5.0.0-oclif4.5](v5.0.0-oclif4.4...v5.0.0-oclif4.5) (2026-03-27) ### Bug Fixes * update dependencies ([636fdfc](636fdfc))

pviti added the released on @oclif4 label Mar 27, 2026

pviti and others added 2 commits April 27, 2026 16:09

Merge branch 'main' into oclif4

e068f5d

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Oclif4#116

Oclif4#116
pviti wants to merge 19 commits into
mainfrom
oclif4

pviti commented Mar 27, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Copilot AI Mar 27, 2026

Uh oh!

Copilot AI Mar 27, 2026

Uh oh!

Copilot AI Mar 27, 2026

Uh oh!

Copilot AI Mar 27, 2026

Uh oh!

Copilot AI Mar 27, 2026

Uh oh!

Copilot AI Mar 27, 2026

Uh oh!

Copilot AI Mar 27, 2026

Uh oh!

Uh oh!

Copilot AI Mar 27, 2026

Uh oh!

pviti commented Mar 27, 2026

Uh oh!

pviti commented Apr 27, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

	const { stdout } = await runCommand<{ name: string }>(['imports:noc'])
	const { stdout } = await runCommand(['imports:noc'])

	describe('imports:create', () => {
	describe('imports:noc', () => {

	describe('imports:details', () => {
	describe('imports:noc', () => {

		describe('imports:types', () => {
		it('runs NoC', async () => {

	describe('imports:group', () => {
	describe('imports:noc', () => {

		describe('imports:list', () => {
		it('runs NoC', async () => {

Conversation

pviti commented Mar 27, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Copilot AI Mar 27, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 27, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 27, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 27, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 27, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 27, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 27, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Copilot AI Mar 27, 2026

Choose a reason for hiding this comment

Uh oh!

pviti commented Mar 27, 2026

Uh oh!

pviti commented Apr 27, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants