Skip to content

DX - 9363 - fix(flags)!: remove --ignore from auth:tokens:remove and -n from config:set:region #2628

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

Open
cs-raj wants to merge 3 commits into
base: v2-dev
Choose a base branch
from
+7 −10
Open

DX - 9363 - fix(flags)!: remove --ignore from auth:tokens:remove and -n from config:set:region #2628

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
8c84220
fix(flags): remove command-specific short flags from auth and config …
cs-raj Jul 1, 2026
100838c
removed ignore flag from auth:token:remove command
cs-raj Jul 1, 2026
5c0b758
Merge branch 'v2-dev' into fix/DX-9363
cs-raj Jul 1, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 7 additions & 9 deletions packages/contentstack-auth/src/commands/auth/tokens/remove.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ export default class TokensRemoveCommand extends BaseCommand<typeof TokensRemove
static examples = ['$ csdx auth:tokens:remove', '$ csdx auth:tokens:remove -a <alias>'];
static flags: FlagInput = {
alias: flags.string({ char: 'a', description: 'Alias (name) of the token to delete.' }),
ignore: flags.boolean({ char: 'i', description: 'Ignores if the token is not present.' }),
};

async run(): Promise<any> {
Expand All @@ -17,21 +16,20 @@ export default class TokensRemoveCommand extends BaseCommand<typeof TokensRemove
log.debug('Token removal flags parsed.', {...this.contextDetails, flags: removeTokenFlags });

const alias = removeTokenFlags.alias;
const ignore = removeTokenFlags.ignore;
log.debug('Token removal parameters set.', {...this.contextDetails, alias, ignore });

log.debug('Token removal parameters set.', {...this.contextDetails, alias });

try {
log.debug('Retrieving token from configuration.', {...this.contextDetails, alias });
const token = configHandler.get(`tokens.${alias}`);
log.debug('Token retrieved from configuration.', {...this.contextDetails, hasToken: !!token, tokenType: token?.type });

const tokens = configHandler.get('tokens');
log.debug('All tokens retrieved from configuration.', {...this.contextDetails, tokenCount: tokens ? Object.keys(tokens).length : 0 });

const tokenOptions: Array<string> = [];
if (token || ignore) {
log.debug('Token found, or ignore flag set.', {...this.contextDetails, hasToken: !!token, ignore });

if (token) {
log.debug('Token found.', {...this.contextDetails, hasToken: !!token });
configHandler.delete(`tokens.${alias}`);
log.debug('Token removed from configuration.', {...this.contextDetails, alias });
return cliux.success(`CLI_AUTH_TOKENS_REMOVE_SUCCESS`);
Comment on lines +31 to 35
Expand Down
1 change: 0 additions & 1 deletion packages/contentstack-config/src/commands/config/set/region.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,6 @@ export default class RegionSetCommand extends BaseCommand<typeof RegionSetComman
dependsOn: ['cda', 'cma', 'name'],
}),
name: _flags.string({
char: 'n',
description: 'Name for the region, if this flag is added then cda, cma and ui-host flags are required',
dependsOn: ['cda', 'cma', 'ui-host'],
}),
Expand Down
Loading