From e6f7ab105df8b5dd35969d9f942ee58b7d0ded01 Mon Sep 17 00:00:00 2001
From: Gabriel Dugny <gabriel.dugny@believe.com>
Date: Mon, 16 Feb 2026 17:38:48 +0100
Subject: [PATCH] fix: Revert some un-necessary allowExcessArguments

Signed-off-by: Gabriel Dugny <gabriel.dugny@believe.com>
---
 packages/cli/src/lib/role.test.ts         | 7 +++----
 packages/repo-tools/src/commands/index.ts | 3 ---
 2 files changed, 3 insertions(+), 7 deletions(-)

diff --git a/packages/cli/src/lib/role.test.ts b/packages/cli/src/lib/role.test.ts
index b56719ad7d..76f51df06f 100644
--- a/packages/cli/src/lib/role.test.ts
+++ b/packages/cli/src/lib/role.test.ts
@@ -29,11 +29,10 @@ jest.mock('./paths', () => ({
 }));
 
 describe('findRoleFromCommand', () => {
-  function mkCommand(args: string) {
+  function mkCommand(args?: string) {
     const parsed = new Command()
-      .allowExcessArguments(true)
       .option('--role <role>', 'test role')
-      .parse(['node', 'entry.js', ...args.split(' ')]) as Command;
+      .parse(args?.split(' ') ?? [], { from: 'user' });
     return parsed.opts();
   }
 
@@ -49,7 +48,7 @@ describe('findRoleFromCommand', () => {
   });
 
   it('provides role info by role', async () => {
-    await expect(findRoleFromCommand(mkCommand(''))).resolves.toEqual(
+    await expect(findRoleFromCommand(mkCommand())).resolves.toEqual(
       'web-library',
     );
 
diff --git a/packages/repo-tools/src/commands/index.ts b/packages/repo-tools/src/commands/index.ts
index 47849778b9..dab7d6df0d 100644
--- a/packages/repo-tools/src/commands/index.ts
+++ b/packages/repo-tools/src/commands/index.ts
@@ -205,7 +205,6 @@ export function registerCommands(program: Command) {
   program
     .command('type-deps')
     .description('Find inconsistencies in types of all packages and plugins')
-    .allowExcessArguments(true)
     .action(lazy(() => import('./type-deps/type-deps'), 'default'));
 
   program
@@ -214,7 +213,6 @@ export function registerCommands(program: Command) {
       'Ensure your packages are using the correct peer dependency format.',
     )
     .option('--fix', 'Fix the issues found')
-    .allowExcessArguments(true)
     .action(lazy(() => import('./peer-deps/peer-deps'), 'default'));
 
   program
@@ -228,7 +226,6 @@ export function registerCommands(program: Command) {
       'CI run checks that there are no changes to catalog-info.yaml files',
     )
     .description('Create or fix info yaml files for all backstage packages')
-    .allowExcessArguments(true)
     .action(
       lazy(
         () => import('./generate-catalog-info/generate-catalog-info'),