Skip to content

Conversation

@aman19K
Copy link
Contributor

@aman19K aman19K commented Nov 3, 2025

RCA:-
The app:create command had an issue where users couldn't properly choose the App Name during the interactive flow. Currently user can override only through flag. But according to documentation we're supporting prompt based approach as well.

Fix:-
If the user doesn’t provide a name flag, they’ll be prompted with the default app name. To use the default boilerplate name, they can simply press Enter. Alternatively, they can enter a custom name of their choice. Once the name is entered, it will be validated according to the existing logic.

@github-actions
Copy link

github-actions bot commented Nov 3, 2025

Coverage report for commit: 9e5c755
File: coverage/clover.xml

Cover ┌─────────────────────────┐ Freq.
   0% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  10% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  20% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  30% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  40% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  50% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  60% │ ███░░░░░░░░░░░░░░░░░░░░ │  4.2%
  70% │ █████████░░░░░░░░░░░░░░ │ 16.7%
  80% │ █████████░░░░░░░░░░░░░░ │ 16.7%
  90% │ █████████░░░░░░░░░░░░░░ │ 16.7%
 100% │ ███████████████████████ │ 45.8%
      └─────────────────────────┘
 *Legend:* █ = Current Distribution 
Summary - Lines: 79.58% | Methods: 76.13% | Branches: 57.27%
FilesLinesMethodsBranches
src
   app-cli-base-command.ts80.00%100.00%50.00%
   base-command.ts80.00%80.00%56.25%
src/commands/app
   create.ts83.11%95.00%67.44%
   delete.ts87.50%100.00%73.47%
   deploy.ts62.79%70.00%50.55%
   get.ts100.00%100.00%74.07%
   index.ts66.67%--
   install.ts90.00%100.00%57.80%
   reinstall.ts90.24%100.00%57.02%
   uninstall.ts100.00%100.00%63.41%
   update.ts71.05%50.00%75.76%
src/config
   index.ts100.00%100.00%100.00%
src/factories
   uninstall-app-factory.ts100.00%100.00%100.00%
src/graphql
   queries.ts100.00%100.00%100.00%
src/messages
   index.ts100.00%100.00%50.00%
src/strategies
   uninstall-all.ts100.00%100.00%100.00%
   uninstall-selected.ts91.67%100.00%50.00%
src/types
   app.ts100.00%100.00%100.00%
   index.ts100.00%100.00%100.00%
src/util
   common-utils.ts69.93%61.29%27.35%
   fs.ts95.24%100.00%83.33%
   index.ts100.00%100.00%100.00%
   inquirer.ts77.40%75.00%57.55%
   log.ts71.93%66.67%48.15%

🤖 comment via lucassabreu/comment-coverage-clover

@aman19K aman19K marked this pull request as ready for review November 3, 2025 12:05
@aman19K aman19K requested a review from a team as a code owner November 3, 2025 12:05
@aman19K aman19K merged commit bbd09f6 into development Nov 5, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants