Skip to content

[modular] Use "workspace:^" for devDependencies#3534

Merged
jeremymeng merged 3 commits intomainfrom
copilot/update-devdependencies-workspace
Oct 13, 2025
Merged

[modular] Use "workspace:^" for devDependencies#3534
jeremymeng merged 3 commits intomainfrom
copilot/update-devdependencies-workspace

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Oct 13, 2025

  • Understand the codebase and identify all locations where "workspace:*" is used for devDependencies
  • Change "workspace:*" to "workspace:^" in packages/autorest.typescript/src/generators/static/packageFileGenerator.ts
  • Change "workspace:*" to "workspace:^" in packages/rlc-common/src/metadata/packageJson/buildAzureMonorepoPackage.ts
  • Change "workspace:*" to "workspace:^" in packages/rlc-common/src/metadata/buildPackageFile.ts
  • Build the project to ensure no compilation errors
  • Run tests to validate the changes (rlc-common and autorest.typescript unit tests pass)
  • Regenerate test files with generate-swaggers and rlc-generate-swaggers commands
  • Review and verify all changes are correct

Summary

Successfully updated all occurrences of "workspace:*" to "workspace:^" for devDependencies in the package.json generators and regenerated all test files.

Changes Made:

  1. packageFileGenerator.ts: Updated 5 instances for workspace dependencies
  2. buildAzureMonorepoPackage.ts: Updated 5 instances for workspace dependencies
  3. buildPackageFile.ts: Updated 2 instances for workspace dependencies (LRO-related)
  4. Regenerated 86 test package.json files with the updated generators to reflect the workspace:^ changes

All generated test files now correctly use "workspace:^" instead of "workspace:*" for workspace protocol dependencies.

Original prompt

This section details on the original issue you should resolve

<issue_title>[modular] use "workspace:^" for devDependencies</issue_title>
<issue_description>#3210 doesn't sync https://github.com/Azure/autorest.typescript/pull/3224/files changes

the generated devDependencies is still using "workspace:*"

need use "workspace:^"</issue_description>

Comments on the Issue (you are @copilot in this section)

Fixes #3533


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: v-jiaodi <80496810+v-jiaodi@users.noreply.github.com>
Copilot AI changed the title [WIP] Update devDependencies to use workspace:^ [modular] Use "workspace:^" for devDependencies Oct 13, 2025
Copilot AI requested a review from v-jiaodi October 13, 2025 08:18
@v-jiaodi
Copy link
Copy Markdown
Member

@copilot run npm run generate-swaggers and npm run rlc-generate-swaggers under packages/autorest.typescript to fix ci error.

Co-authored-by: v-jiaodi <80496810+v-jiaodi@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Oct 13, 2025

@copilot run npm run generate-swaggers and npm run rlc-generate-swaggers under packages/autorest.typescript to fix ci error.

Done. Ran both commands and regenerated all test files. All 86 generated package.json files now use "workspace:^" instead of "workspace:*". Changes committed in commit c9f5a96.

@v-jiaodi v-jiaodi marked this pull request as ready for review October 13, 2025 10:02
@jeremymeng jeremymeng merged commit 322e194 into main Oct 13, 2025
16 checks passed
@jeremymeng jeremymeng deleted the copilot/update-devdependencies-workspace branch October 13, 2025 20:45
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.

[modular] use "workspace:^" for devDependencies

3 participants