Skip to content

feat: add project ID and resource prefix to webiny info output#4937

Merged
adrians5j merged 5 commits intonextfrom
copilot/add-project-id-and-resource-prefix
Feb 13, 2026
Merged

feat: add project ID and resource prefix to webiny info output#4937
adrians5j merged 5 commits intonextfrom
copilot/add-project-id-and-resource-prefix

Conversation

Copy link
Contributor

Copilot AI commented Feb 13, 2026

  • Import GetProjectIdService and GetPulumiResourceNamePrefix abstractions from @webiny/project
  • Inject these services into InfoCommand constructor
  • Pass services to PrintInfoForEnv through its DI interface
  • Update PrintInfoForEnv to fetch and display project ID and resource prefix
  • Fix DeployCommand to inject and pass required services
  • Refactor to use ProjectSdk methods (getProjectId, getPulumiResourceNamePrefix) instead of abstractions
  • Test the changes manually (no test infrastructure exists)
  • Run code review (no issues found)
  • Run CodeQL security scan (no issues found)

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: adrians5j <5121148+adrians5j@users.noreply.github.com>
Copilot AI changed the title [WIP] Add project ID and resource prefix to webiny info command feat: add project ID and resource prefix to webiny info output Feb 13, 2026
Copilot AI requested a review from adrians5j February 13, 2026 09:10
@adrians5j adrians5j marked this pull request as ready for review February 13, 2026 09:29
@Pavel910 Pavel910 added this to the 6.0.0 milestone Feb 13, 2026
Copy link
Member

@adrians5j adrians5j left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✖ @webiny/cli-core
src/features/DeployCommand/DeployCommand.ts(147,65): error TS2345: Argument of type '{ getProjectSdkService: IGetProjectSdkService; uiService: IUiService; }' is not assignable to parameter of type 'IPrintInfoForEnvDi'.
Type '{ getProjectSdkService: IGetProjectSdkService; uiService: IUiService; }' is missing the following properties from type 'IPrintInfoForEnvDi': getProjectIdService, getPulumiResourceNamePrefix @copilot

Co-authored-by: adrians5j <5121148+adrians5j@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Feb 13, 2026

✖ @webiny/cli-core
src/features/DeployCommand/DeployCommand.ts(147,65): error TS2345: Argument of type '{ getProjectSdkService: IGetProjectSdkService; uiService: IUiService; }' is not assignable to ...

Fixed in commit 08a3490. Updated DeployCommand to inject and pass GetProjectIdService and GetPulumiResourceNamePrefix to PrintInfoForEnv.


const { env, variant } = params;

const projectId = await getProjectIdService.execute();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do not use abstractions here.... projectSDK itself has these methods exposed I believe

getPulumiResourceNamePrefix
getProjectId

@copilot

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Refactored in commit 7d5d9a9. Now using projectSdk.getProjectId() and projectSdk.getPulumiResourceNamePrefix() directly instead of injecting service abstractions.

Co-authored-by: adrians5j <5121148+adrians5j@users.noreply.github.com>
@adrians5j adrians5j merged commit 319f33e into next Feb 13, 2026
17 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