Skip to content

Commit d4e0151

Browse files
Merge pull request #2446 from contentstack/fix/dx-4439-v2-dev-interactive-mode-messages
Fix/dx 4439 v2 dev interactive mode messages
2 parents 6f90ffe + b403892 commit d4e0151

File tree

5 files changed

+457
-413
lines changed

5 files changed

+457
-413
lines changed

packages/contentstack-auth/messages/index.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@
1818
"CLI_AUTH_LOGOUT_DESCRIPTION": "User session logout",
1919
"CLI_AUTH_LOGOUT_FLAG_FORCE": "Force logging out for skipping the confirmation",
2020
"CLI_AUTH_LOGOUT_ALREADY": "You're already logged out",
21+
"CLI_AUTH_LOGOUT_CANCELLED": "Log out cancelled",
2122
"CLI_AUTH_LOGOUT_NO_AUTHORIZATIONS": "No authorizations found",
2223
"CLI_AUTH_LOGOUT_NO_AUTHORIZATIONS_USER": "No authorizations found for current user",
2324
"CLI_AUTH_WHOAMI_LOGGED_IN_AS": "You are currently logged in with email:",

packages/contentstack-auth/src/commands/auth/logout.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,10 @@ export default class LogoutCommand extends BaseCommand<typeof LogoutCommand> {
8484
confirm,
8585
isAuthenticated: oauthHandler.isAuthenticated(),
8686
});
87-
log.success(messageHandler.parse('CLI_AUTH_LOGOUT_ALREADY'), this.contextDetails);
87+
const messageKey = confirm === false && oauthHandler.isAuthenticated()
88+
? 'CLI_AUTH_LOGOUT_CANCELLED'
89+
: 'CLI_AUTH_LOGOUT_ALREADY';
90+
log.success(messageHandler.parse(messageKey), this.contextDetails);
8891
}
8992
} catch (error) {
9093
log.debug('Logout failed.', { ...this.contextDetails, error: error.message });

packages/contentstack/package.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -157,6 +157,7 @@
157157
],
158158
"hooks": {
159159
"prerun": [
160+
"./lib/hooks/prerun/init-context-for-command",
160161
"./lib/hooks/prerun/default-rate-limit-check",
161162
"./lib/hooks/prerun/latest-version-warning"
162163
],
Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,27 @@
1+
import {
2+
cliux,
3+
messageHandler,
4+
managementSDKInitiator,
5+
marketplaceSDKInitiator,
6+
} from '@contentstack/cli-utilities';
7+
import { CsdxContext } from '../../utils';
8+
9+
/**
10+
* When an invalid command is corrected (e.g. loginasda → login), init ran with the invalid
11+
* command so context.messageFilePath was never set. Re-build context and re-init utilities
12+
* for the actual command so i18n prompts show human-readable text.
13+
*/
14+
export default async function (opts: {
15+
Command?: { id?: string };
16+
config?: any;
17+
}): Promise<void> {
18+
const config = opts?.config ?? this.config;
19+
const commandId = opts?.Command?.id;
20+
if (!config?.context?.messageFilePath && commandId) {
21+
config.context = new CsdxContext({ id: commandId }, config);
22+
messageHandler.init(config.context);
23+
cliux.init(config.context);
24+
managementSDKInitiator.init(config.context);
25+
marketplaceSDKInitiator.init(config.context);
26+
}
27+
}

0 commit comments

Comments
 (0)