Skip to content

Commit ac5d7b6

Browse files
fix: dedupe environments get by using shared parseKeysFromArgs
1 parent c44619d commit ac5d7b6

1 file changed

Lines changed: 2 additions & 12 deletions

File tree

  • src/commands/environments

src/commands/environments/get.ts

Lines changed: 2 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ import {
77
import { EnvironmentPromptResult, environmentPrompt } from '../../ui/prompts'
88
import Base from '../base'
99
import { batchRequests } from '../../utils/batchRequests'
10+
import { parseKeysFromArgs } from '../../utils/parseKeysFromArgs'
1011

1112
export default class DetailedEnvironments extends Base {
1213
static hidden = false
@@ -40,18 +41,7 @@ export default class DetailedEnvironments extends Base {
4041
const { headless, project } = flags
4142
await this.requireProject(project, headless)
4243

43-
// Handle positional arguments - they take precedence over --keys flag
44-
let keys: string[] | undefined
45-
if (argv && argv.length > 0) {
46-
// Collect all positional arguments and split any comma-separated values
47-
keys = argv.flatMap((arg) => String(arg).split(','))
48-
} else if (args.keys) {
49-
// Handle the first positional argument if provided
50-
keys = args.keys.split(',')
51-
} else if (flags.keys) {
52-
// Fall back to --keys flag
53-
keys = flags.keys.split(',')
54-
}
44+
const keys = parseKeysFromArgs(args, argv, flags)
5545

5646
if (keys && keys.length > 0) {
5747
const environments = await batchRequests(keys, (key) =>

0 commit comments

Comments
 (0)