nebius/SWE-rebench-V2-PRs huggingface dataset:
Repository: ember-cli/ember-cli
Pull request #9938 (2022-06-12 08:16:16)

[INTERNAL] Fix internal `sequence` util
Noticed this util wasn't working as documented.
Mainly the returned `results` array was incorrect.

Closes #9862.


Patch:
diff --git a/lib/utilities/sequence.js b/lib/utilities/sequence.js
index 23f472877..8b5df8b43 100644
--- a/lib/utilities/sequence.js
+++ b/lib/utilities/sequence.js
@@ -27,14 +27,16 @@
  * @return Promise<Array>
  *
  */
-module.exports = function sequence(tasks) {
+module.exports = async function sequence(tasks) {
   let length = tasks.length;
-  let current = Promise.resolve();
-  let results = new Array(length);
+  let results = [];
 
   for (let i = 0; i < length; ++i) {
-    current = results[i] = current.then(tasks[i]);
+    let task = tasks[i];
+    let prevResult = results[i - 1];
+
+    results.push(typeof task === 'function' ? await task(prevResult) : await task);
   }
 
-  return Promise.all(results);
+  return results;
 };


Test patch:
diff --git a/tests/unit/utilities/sequence-test.js b/tests/unit/utilities/sequence-test.js
new file mode 100644
index 000000000..bf33922e6
--- /dev/null
+++ b/tests/unit/utilities/sequence-test.js
@@ -0,0 +1,18 @@
+'use strict';
+
+const { expect } = require('chai');
+const sequence = require('../../../lib/utilities/sequence');
+
+describe('sequence', function () {
+  it('it works', async function () {
+    let results = await sequence([
+      Promise.resolve(1),
+      () => Promise.resolve(2),
+      3,
+      () => new Promise((resolve) => setTimeout(() => resolve(4), 100)),
+      (prevResult) => prevResult + 1,
+    ]);
+
+    expect(results).to.deep.equal([1, 2, 3, 4, 5]);
+  });
+});
Repository: ember-cli/ember-cli
Pull request #10010 (2022-09-07 15:50:17)

[BUGFIX] Handle rebuild failures without exiting
Cherry picking https://github.com/ember-cli/ember-cli/pull/9987 into lts-4-4.


Patch:
diff --git a/lib/models/builder.js b/lib/models/builder.js
index 16b86a417..12e756203 100644
--- a/lib/models/builder.js
+++ b/lib/models/builder.js
@@ -221,6 +221,10 @@ class Builder extends CoreObject {
     } catch (error) {
       await this.processAddonBuildSteps('buildError', error);
 
+      // Mark this as a builder error so the watcher knows it has been handled
+      // and won't re-throw it
+      error.isBuilderError = true;
+
       throw error;
     } finally {
       clearInterval(uiProgressIntervalID);
diff --git a/lib/tasks/server/middleware/history-support/index.js b/lib/tasks/server/middleware/history-support/index.js
index 3d31dc14a..0635fa461 100644
--- a/lib/tasks/server/middleware/history-support/index.js
+++ b/lib/tasks/server/middleware/history-support/index.js
@@ -49,7 +49,16 @@ class HistorySupportAddon {
 
     app.use(async (req, _, next) => {
       try {
-        const results = await watcher;
+        let results;
+        try {
+          results = await watcher;
+        } catch (e) {
+          // This means there was a build error, so we won't actually be serving
+          // index.html, and we have nothing to do. We have to catch it here,
+          // though, or it will go uncaught and cause the process to exit.
+          return;
+        }
+
         if (this.shouldHandleRequest(req, options)) {
           let assetPath = req.path.slice(baseURL.length);
           let isFile = false;


Test patch:
diff --git a/tests/unit/models/builder-test.js b/tests/unit/models/builder-test.js
index f00f5f180..d92af8566 100644
--- a/tests/unit/models/builder-test.js
+++ b/tests/unit/models/builder-test.js
@@ -478,6 +478,20 @@ describe('models/builder.js', function () {
       await expect(builder.build()).to.be.rejected;
       expect(hooksCalled).to.deep.equal(['preBuild', 'build', 'postBuild', 'outputReady', 'buildError']);
     });
+
+    it('sets `isBuilderError` on handled addon errors', async function () {
+      addon.postBuild = function () {
+        return Promise.reject(new Error('preBuild Error'));
+      };
+
+      let error;
+      try {
+        await builder.build();
+      } catch (e) {
+        error = e;
+      }
+      expect(error).to.haveOwnProperty('isBuilderError', true);
+    });
   });
 
   describe('fallback from broccoli 2 to broccoli-builder', function () {
Repository: ember-cli/ember-cli
Pull request #10107 (2022-12-11 15:08:44)

[INTERNAL] Remove old `uninstall:npm` command
This command was already "removed" years ago, i.e. using it would just log an error.
I think we can completely remove it now, which would log the default error for unknown commands.


Patch:
diff --git a/lib/commands/uninstall-npm.js b/lib/commands/uninstall-npm.js
deleted file mode 100644
index 6633777d2..000000000
--- a/lib/commands/uninstall-npm.js
+++ /dev/null
@@ -1,19 +0,0 @@
-'use strict';
-
-const Command = require('../models/command');
-const SilentError = require('silent-error');
-
-module.exports = Command.extend({
-  name: 'uninstall:npm',
-  description: 'Npm package uninstall are now managed by the user.',
-  works: 'insideProject',
-  skipHelp: true,
-
-  anonymousOptions: ['<package-names...>'],
-
-  run() {
-    let err = 'This command has been removed. Please use `npm uninstall ';
-    err += '<packageName> --save-dev` instead.';
-    return Promise.reject(new SilentError(err));
-  },
-});


Test patch:
diff --git a/tests/unit/commands/uninstall-npm-test.js b/tests/unit/commands/uninstall-npm-test.js
deleted file mode 100644
index 31333c341..000000000
--- a/tests/unit/commands/uninstall-npm-test.js
+++ /dev/null
@@ -1,38 +0,0 @@
-'use strict';
-
-const expect = require('../../chai').expect;
-const MockProject = require('../../helpers/mock-project');
-const commandOptions = require('../../factories/command-options');
-const UninstallNpmCommand = require('../../../lib/commands/uninstall-npm');
-
-describe('uninstall:npm command', function () {
-  let command;
-
-  let msg = 'This command has been removed. Please use `npm uninstall ' + '<packageName> --save-dev` instead.';
-
-  beforeEach(function () {
-    let project = new MockProject();
-
-    project.isEmberCLIProject = function () {
-      return true;
-    };
-
-    let options = commandOptions({
-      project,
-    });
-
-    command = new UninstallNpmCommand(options);
-  });
-
-  it('throws a friendly silent error with no args', function () {
-    return expect(command.validateAndRun([])).to.be.rejected.then((error) => {
-      expect(error.message).to.equal(msg, 'expect error to have a helpful message');
-    });
-  });
-
-  it('throws a friendly silent error with args', function () {
-    return expect(command.validateAndRun(['moment', 'lodash'])).to.be.rejected.then((error) => {
-      expect(error.message).to.equal(msg, 'expect error to have a helpful message');
-    });
-  });
-});
Repository: ember-cli/ember-cli
Pull request #10108 (2022-12-12 16:17:52)

[BUGFIX release] Correctly instantiate server watcher
Fixes:

```
instantiate Watcher with (await Watcher.build()).watcher, not new Watcher()
```

when running:

```
ember s
```

in a project that has a `server` directory.

Similar to https://github.com/ember-cli/ember-cli/pull/10101.


Patch:
diff --git a/lib/models/server-watcher.js b/lib/models/server-watcher.js
index d2e4677e6..ad0fc0e13 100644
--- a/lib/models/server-watcher.js
+++ b/lib/models/server-watcher.js
@@ -3,11 +3,13 @@
 const Watcher = require('./watcher');
 
 module.exports = class ServerWatcher extends Watcher {
-  constructor(options, build) {
-    super(options, build);
+  static async build(options, build) {
+    let { watcher: instance } = await super.build(options, build);
 
-    this.watcher.on('add', this.didAdd.bind(this));
-    this.watcher.on('delete', this.didDelete.bind(this));
+    instance.watcher.on('add', instance.didAdd.bind(instance));
+    instance.watcher.on('delete', instance.didDelete.bind(instance));
+
+    return { watcher: instance };
   }
 
   constructBroccoliWatcher(options) {
diff --git a/lib/tasks/serve.js b/lib/tasks/serve.js
index e26bab3f1..60c1cefad 100644
--- a/lib/tasks/serve.js
+++ b/lib/tasks/serve.js
@@ -69,12 +69,14 @@ class ServeTask extends Task {
     let serverRoot = './server';
     let serverWatcher = null;
     if (fs.existsSync(serverRoot)) {
-      serverWatcher = new ServerWatcher({
-        ui: this.ui,
-        analytics: this.analytics,
-        watchedDir: path.resolve(serverRoot),
-        options,
-      });
+      serverWatcher = (
+        await ServerWatcher.build({
+          ui: this.ui,
+          analytics: this.analytics,
+          watchedDir: path.resolve(serverRoot),
+          options,
+        })
+      ).watcher;
     }
 
     let expressServer =


Test patch:
diff --git a/tests/unit/models/server-watcher-test.js b/tests/unit/models/server-watcher-test.js
index ad6544cb5..541bb99de 100644
--- a/tests/unit/models/server-watcher-test.js
+++ b/tests/unit/models/server-watcher-test.js
@@ -17,10 +17,17 @@ describe('Server Watcher', function () {
     analytics = new MockAnalytics();
     watcher = new MockServerWatcher();
 
-    await ServerWatcher.build({
+    class ServerWatcherMock extends ServerWatcher {
+      setupBroccoliWatcher() {
+        this.watcher = watcher;
+
+        return super.setupBroccoliWatcher(...arguments);
+      }
+    }
+
+    await ServerWatcherMock.build({
       ui,
       analytics,
-      watcher,
     });
   });
Repository: ember-cli/ember-cli
Pull request #10185 (2023-02-14 20:24:53)

[CLEANUP] Drop support for the `EMBER_CLI_ERROR_ON_INVALID_ADDON` env flag
Having an invalid addon path in `ember-addon.paths` will now always throw an error instead of a warning.


Patch:
diff --git a/lib/models/package-info-cache/package-info.js b/lib/models/package-info-cache/package-info.js
index 4ec686024..690f105a2 100644
--- a/lib/models/package-info-cache/package-info.js
+++ b/lib/models/package-info-cache/package-info.js
@@ -412,11 +412,7 @@ class PackageInfo {
         });
       }
 
-      if (process.env.EMBER_CLI_ERROR_ON_INVALID_ADDON) {
-        throw msg;
-      } else {
-        this.cache.ui.writeWarnLine(msg);
-      }
+      throw msg;
     }
   }
 


Test patch:
diff --git a/tests/fixtures/addon/with-app-styles/package.json b/tests/fixtures/addon/with-app-styles/package.json
index e069a0853..4d70d5adf 100644
--- a/tests/fixtures/addon/with-app-styles/package.json
+++ b/tests/fixtures/addon/with-app-styles/package.json
@@ -4,10 +4,7 @@
   "dependencies": {
     "something-else": "latest"
   },
-  "ember-addon": {
-    "paths": ["./lib/ember-super-button"]
-  },
-    "devDependencies": {
+  "devDependencies": {
     "ember-resolver": "^7.0.0",
     "ember-cli": "latest",
     "ember-random-addon": "latest",
diff --git a/tests/unit/models/package-info-cache/package-info-cache-test.js b/tests/unit/models/package-info-cache/package-info-cache-test.js
index b6a961921..45cfcc648 100644
--- a/tests/unit/models/package-info-cache/package-info-cache-test.js
+++ b/tests/unit/models/package-info-cache/package-info-cache-test.js
@@ -259,18 +259,9 @@ describe('models/package-info-cache/package-info-cache-test.js', function () {
 
       afterEach(function () {
         fixturifyProject.dispose();
-        delete process.env.EMBER_CLI_ERROR_ON_INVALID_ADDON;
       });
 
-      it('shows a warning with invalid ember-addon#path', function () {
-        project.discoverAddons();
-        expect(project.cli.ui.output).to.include(
-          `specifies an invalid, malformed or missing addon at relative path 'lib${path.sep}no-such-path'`
-        );
-      });
-
-      it('throws an error with flag on', function () {
-        process.env.EMBER_CLI_ERROR_ON_INVALID_ADDON = 'true';
+      it('throws an error', function () {
         expect(() => project.discoverAddons()).to.throw(
           /specifies an invalid, malformed or missing addon at relative path 'lib[\\/]no-such-path'/
         );
Repository: ember-cli/ember-cli
Pull request #10192 (2023-02-21 21:29:12)

[BUGFIX] The `addon` command should throw when no addon name is provided
At the moment, when you run the `addon` command _without_ providing an addon name, it triggers the interactive-new flow, because the `addon` command extends the `new` command. According to https://github.com/emberjs/rfcs/pull/638, the `addon` command should never trigger the interactive-new flow. This means we should error when no addon name is provided [like we used to](https://github.com/bertdeblock/ember-cli/blob/5c26dc268f7da33b9ee3519d3ac85c03771783ae/lib/commands/new.js#L56-L60).


Patch:
diff --git a/lib/commands/addon.js b/lib/commands/addon.js
index 95652dcbf..252701f0a 100644
--- a/lib/commands/addon.js
+++ b/lib/commands/addon.js
@@ -1,5 +1,6 @@
 'use strict';
 
+const SilentError = require('silent-error');
 const NewCommand = require('./new');
 
 module.exports = NewCommand.extend({
@@ -30,4 +31,16 @@ module.exports = NewCommand.extend({
   ],
 
   anonymousOptions: ['<addon-name>'],
+
+  run(commandOptions, commandArguments) {
+    let addonName = commandArguments[0];
+
+    if (addonName) {
+      return this._super(commandOptions, commandArguments);
+    }
+
+    return Promise.reject(
+      new SilentError('The `ember addon` command requires a name to be specified. For more details, run `ember help`.')
+    );
+  },
 });


Test patch:
diff --git a/tests/unit/commands/addon-test.js b/tests/unit/commands/addon-test.js
index fc7df2017..0cb772bc8 100644
--- a/tests/unit/commands/addon-test.js
+++ b/tests/unit/commands/addon-test.js
@@ -78,6 +78,12 @@ describe('addon command', function () {
     });
   });
 
+  it('throws when no addon name is provided', async function () {
+    expect(command.validateAndRun([])).to.be.rejectedWith(
+      'The `ember addon` command requires a name to be specified. For more details, run `ember help`.'
+    );
+  });
+
   it('registers blueprint options in beforeRun', function () {
     td.replace(Blueprint, 'lookup', td.function());
Repository: ember-cli/ember-cli
Pull request #10316 (2023-07-19 20:34:58)

[BUGFIX] Remove `auto` as a possible value for `locationType` in `config` declaration
The `auto` location was removed in `ember-source` v5.


Patch:
diff --git a/blueprints/app/files/app/config/environment.d.ts b/blueprints/app/files/app/config/environment.d.ts
index 0c5780482..8a75a8357 100644
--- a/blueprints/app/files/app/config/environment.d.ts
+++ b/blueprints/app/files/app/config/environment.d.ts
@@ -6,7 +6,7 @@ declare const config: {
   environment: string;
   modulePrefix: string;
   podModulePrefix: string;
-  locationType: 'history' | 'hash' | 'none' | 'auto';
+  locationType: 'history' | 'hash' | 'none';
   rootURL: string;
   APP: Record<string, unknown>;
 };


Test patch:
diff --git a/tests/fixtures/addon/typescript/tests/dummy/app/config/environment.d.ts b/tests/fixtures/addon/typescript/tests/dummy/app/config/environment.d.ts
index 1725b438f..91eec535d 100644
--- a/tests/fixtures/addon/typescript/tests/dummy/app/config/environment.d.ts
+++ b/tests/fixtures/addon/typescript/tests/dummy/app/config/environment.d.ts
@@ -6,7 +6,7 @@ declare const config: {
   environment: string;
   modulePrefix: string;
   podModulePrefix: string;
-  locationType: 'history' | 'hash' | 'none' | 'auto';
+  locationType: 'history' | 'hash' | 'none';
   rootURL: string;
   APP: Record<string, unknown>;
 };
diff --git a/tests/fixtures/app/typescript-embroider/app/config/environment.d.ts b/tests/fixtures/app/typescript-embroider/app/config/environment.d.ts
index f0cad8b31..8f58c600d 100644
--- a/tests/fixtures/app/typescript-embroider/app/config/environment.d.ts
+++ b/tests/fixtures/app/typescript-embroider/app/config/environment.d.ts
@@ -6,7 +6,7 @@ declare const config: {
   environment: string;
   modulePrefix: string;
   podModulePrefix: string;
-  locationType: 'history' | 'hash' | 'none' | 'auto';
+  locationType: 'history' | 'hash' | 'none';
   rootURL: string;
   APP: Record<string, unknown>;
 };
diff --git a/tests/fixtures/app/typescript/app/config/environment.d.ts b/tests/fixtures/app/typescript/app/config/environment.d.ts
index f0cad8b31..8f58c600d 100644
--- a/tests/fixtures/app/typescript/app/config/environment.d.ts
+++ b/tests/fixtures/app/typescript/app/config/environment.d.ts
@@ -6,7 +6,7 @@ declare const config: {
   environment: string;
   modulePrefix: string;
   podModulePrefix: string;
-  locationType: 'history' | 'hash' | 'none' | 'auto';
+  locationType: 'history' | 'hash' | 'none';
   rootURL: string;
   APP: Record<string, unknown>;
 };
Repository: ember-cli/ember-cli
Pull request #10345 (2023-08-19 16:09:35)

[BUGFIX beta] App blueprint may not have explicit-any in ember-data types
Discovered over here: https://github.com/embroider-build/addon-blueprint/actions/runs/5912329738/job/16035634804?pr=188

In `--typescript` projects, we use
```
      extends: [
        'plugin:@typescript-eslint/eslint-recommended',
        'plugin:@typescript-eslint/recommended',
      ],
```

which then immediately fails `lint` / `lint:fix` due to the above configs having `@typescript-eslint/no-explicit-any` so we need to disable the rule for ember-data's model registry.


Unblocks: https://github.com/embroider-build/addon-blueprint/pull/188


Patch:
diff --git a/blueprints/app/files/types/ember-data/types/registries/model.d.ts b/blueprints/app/files/types/ember-data/types/registries/model.d.ts
index e7a68fcd0..bdd8c0f17 100644
--- a/blueprints/app/files/types/ember-data/types/registries/model.d.ts
+++ b/blueprints/app/files/types/ember-data/types/registries/model.d.ts
@@ -2,5 +2,6 @@
  * Catch-all for ember-data.
  */
 export default interface ModelRegistry {
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   [key: string]: any;
 }


Test patch:
diff --git a/tests/fixtures/app/typescript-embroider/types/ember-data/types/registries/model.d.ts b/tests/fixtures/app/typescript-embroider/types/ember-data/types/registries/model.d.ts
index e7a68fcd0..bdd8c0f17 100644
--- a/tests/fixtures/app/typescript-embroider/types/ember-data/types/registries/model.d.ts
+++ b/tests/fixtures/app/typescript-embroider/types/ember-data/types/registries/model.d.ts
@@ -2,5 +2,6 @@
  * Catch-all for ember-data.
  */
 export default interface ModelRegistry {
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   [key: string]: any;
 }
diff --git a/tests/fixtures/app/typescript/types/ember-data/types/registries/model.d.ts b/tests/fixtures/app/typescript/types/ember-data/types/registries/model.d.ts
index e7a68fcd0..bdd8c0f17 100644
--- a/tests/fixtures/app/typescript/types/ember-data/types/registries/model.d.ts
+++ b/tests/fixtures/app/typescript/types/ember-data/types/registries/model.d.ts
@@ -2,5 +2,6 @@
  * Catch-all for ember-data.
  */
 export default interface ModelRegistry {
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   [key: string]: any;
 }
Repository: ember-cli/ember-cli
Pull request #10346 (2023-08-19 16:12:45)

[BUGFIX release] App blueprint may not have explicit-any in ember-data types 
5.2 version of: https://github.com/ember-cli/ember-cli/pull/10345

Discovered over here: https://github.com/embroider-build/addon-blueprint/actions/runs/5912329738/job/16035634804?pr=188

In `--typescript` projects, we use
```
      extends: [
        'plugin:@typescript-eslint/eslint-recommended',
        'plugin:@typescript-eslint/recommended',
      ],
```

which then immediately fails `lint` / `lint:fix` due to the above configs having `@typescript-eslint/no-explicit-any` so we need to disable the rule for ember-data's model registry.


Unblocks: https://github.com/embroider-build/addon-blueprint/pull/188


Patch:
diff --git a/blueprints/app/files/types/ember-data/types/registries/model.d.ts b/blueprints/app/files/types/ember-data/types/registries/model.d.ts
index e7a68fcd0..bdd8c0f17 100644
--- a/blueprints/app/files/types/ember-data/types/registries/model.d.ts
+++ b/blueprints/app/files/types/ember-data/types/registries/model.d.ts
@@ -2,5 +2,6 @@
  * Catch-all for ember-data.
  */
 export default interface ModelRegistry {
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   [key: string]: any;
 }


Test patch:
diff --git a/tests/fixtures/app/typescript-embroider/types/ember-data/types/registries/model.d.ts b/tests/fixtures/app/typescript-embroider/types/ember-data/types/registries/model.d.ts
index e7a68fcd0..bdd8c0f17 100644
--- a/tests/fixtures/app/typescript-embroider/types/ember-data/types/registries/model.d.ts
+++ b/tests/fixtures/app/typescript-embroider/types/ember-data/types/registries/model.d.ts
@@ -2,5 +2,6 @@
  * Catch-all for ember-data.
  */
 export default interface ModelRegistry {
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   [key: string]: any;
 }
diff --git a/tests/fixtures/app/typescript/types/ember-data/types/registries/model.d.ts b/tests/fixtures/app/typescript/types/ember-data/types/registries/model.d.ts
index e7a68fcd0..bdd8c0f17 100644
--- a/tests/fixtures/app/typescript/types/ember-data/types/registries/model.d.ts
+++ b/tests/fixtures/app/typescript/types/ember-data/types/registries/model.d.ts
@@ -2,5 +2,6 @@
  * Catch-all for ember-data.
  */
 export default interface ModelRegistry {
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   [key: string]: any;
 }
Repository: ember-cli/ember-cli
Pull request #10425 (2024-01-08 21:43:40)

fix GitHub Action floating deps scenario for PNPM
The _Floating Dependencies_ scenario is broken for a newly created Ember addon using PNPM. This change fixes it.

It seems that `wyvox/action-setup-pnpm`'s `no-lockfile` option is not working as expected. It throws the following error:

> Error: Dependencies lock file is not found in /home/runner/work/test-ember-addon-with-pnpm/test-ember-addon-with-pnpm. Supported file patterns: pnpm-lock.yaml

The option has been removed in `wyvox/action-setup-pnpm@v3`. So upgrading is not an option. But using `--no-lockfile` flag of PNPM is the recommended upgrade path anyways: https://github.com/wyvox/action-setup-pnpm/pull/3

Please find a demonstration of the bug here: https://github.com/jelhan/test-ember-addon-with-pnpm/actions/runs/7453402485/job/20278700434

A demonstration that the fix work could be found here: https://github.com/jelhan/test-ember-addon-with-pnpm/pull/1

@NullVoxPopuli would be great if you could review. I think you are the expert here as you implemented the PNPM support in Ember CLI _and_ maintain `wyvox/action-setup-pnpm`.


Patch:
diff --git a/blueprints/addon/files/.github/workflows/ci.yml b/blueprints/addon/files/.github/workflows/ci.yml
index 30dd51d8d..67d5e5bec 100644
--- a/blueprints/addon/files/.github/workflows/ci.yml
+++ b/blueprints/addon/files/.github/workflows/ci.yml
@@ -44,7 +44,7 @@ jobs:
       - uses: wyvox/action-setup-pnpm@v2
         with:
           node-version: 18
-          no-lockfile: true<% } else { %>
+          args: '--no-lockfile'<% } else { %>
       - uses: actions/setup-node@v3
         with:
           node-version: 18


Test patch:
diff --git a/tests/fixtures/addon/pnpm/.github/workflows/ci.yml b/tests/fixtures/addon/pnpm/.github/workflows/ci.yml
index 99884177f..dfa740db8 100644
--- a/tests/fixtures/addon/pnpm/.github/workflows/ci.yml
+++ b/tests/fixtures/addon/pnpm/.github/workflows/ci.yml
@@ -37,7 +37,7 @@ jobs:
       - uses: wyvox/action-setup-pnpm@v2
         with:
           node-version: 18
-          no-lockfile: true
+          args: '--no-lockfile'
       - name: Run Tests
         run: pnpm test:ember