Skip to content

Fix/node lockfiles#203

Closed
rido-min wants to merge 3 commits intomainfrom
fix/node-lockfiles
Closed

Fix/node lockfiles#203
rido-min wants to merge 3 commits intomainfrom
fix/node-lockfiles

Conversation

@rido-min
Copy link
Copy Markdown
Member

@rido-min rido-min commented Jul 7, 2025

This pull request makes several updates across multiple sample projects to improve package management, streamline build processes, and enhance code functionality. The most significant changes involve replacing npm install with npm ci for consistent dependency installation, updating dependencies in package.json, and refining the GetWeatherTool implementation in the weather agent sample.

Package Management Updates:

  • Replaced npm install with npm ci in the prebuild scripts across all package.json files to ensure clean and reliable dependency installation. (samples/basic/cards/nodejs/package.json [1] samples/basic/copilotstudio-client/nodejs/package.json [2] samples/basic/empty-agent/nodejs/package.json [3] samples/basic/weather-agent/nodejs/package.json [4] samples/complex/copilotstudio-skill/nodejs/package.json [5]

  • Removed package-lock=false from .npmrc files to allow the generation of package-lock.json for better dependency tracking. (samples/basic/cards/nodejs/.npmrc [1] samples/basic/copilotstudio-client/nodejs/.npmrc [2] samples/basic/empty-agent/nodejs/.npmrc [3] samples/complex/copilotstudio-skill/nodejs/.npmrc [4]

Dependency Updates:

  • Updated the version of @microsoft/agents-copilotstudio-client dependency in samples/basic/copilotstudio-client/nodejs/package.json from ^0.4.0 to ^0.5.12-g2d752e9b13 to incorporate the latest features and fixes. (samples/basic/copilotstudio-client/nodejs/package.json samples/basic/copilotstudio-client/nodejs/package.jsonL11-R18)

Code Enhancements:

Miscellaneous:

  • Removed package-lock.json from .gitignore to ensure the lock file is tracked in version control. (samples/.gitignore samples/.gitignoreL229)

@rido-min rido-min marked this pull request as ready for review July 7, 2025 15:42
@rido-min
Copy link
Copy Markdown
Member Author

rido-min commented Jul 7, 2025

overseed by #201

@rido-min rido-min closed this Jul 7, 2025
@tracyboehrer tracyboehrer deleted the fix/node-lockfiles branch March 5, 2026 13:00
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.

1 participant