Skip to content

Update tooling deps: npm/json and devcontainer lock#4534

Closed
MasterTLF wants to merge 2 commits into
microsoft:mainfrom
MasterTLF:mastertlf-install-latest-typescript
Closed

Update tooling deps: npm/json and devcontainer lock#4534
MasterTLF wants to merge 2 commits into
microsoft:mainfrom
MasterTLF:mastertlf-install-latest-typescript

Conversation

@MasterTLF

@MasterTLF MasterTLF commented Jul 4, 2026

Copy link
Copy Markdown

Analysis

The session updated local runtime/tooling versions (Node via nvm, npm/json via npm), but repository-tracked tooling deps in _scripts still reflected older state and had no lockfile.

To make the tooling update reproducible in this repo, dependency versions and lock data need to be captured in source control.

Fix

Added json as a devDependency in _scripts/package.json, generated _scripts/package-lock.json, and added .devcontainer/devcontainer-lock.json to pin devcontainer feature resolutions.

Copilot Checklist

I successfully ran these commands at the end of my session, and they completed without error:

  • npx hereby build
  • npx hereby test
  • npx hereby lint
  • npx hereby format

Install latest npm and json in _scripts and add the devcontainer feature lockfile.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 4, 2026 19:51

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR aims to make local tooling/devcontainer setup more reproducible by capturing dependency/feature resolution state in-repo (devcontainer feature lock + _scripts npm deps/lock).

Changes:

  • Added .devcontainer/devcontainer-lock.json to pin resolved devcontainer feature versions/digests.
  • Updated _scripts/package.json to include json and npm as devDependencies.
  • Added _scripts/package-lock.json to lock _scripts dependency resolution.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 3 comments.

File Description
.devcontainer/devcontainer-lock.json New lockfile pinning devcontainer feature resolutions.
_scripts/package.json Adds _scripts devDependencies (json, npm).
_scripts/package-lock.json New lockfile capturing _scripts dependency graph.
Files not reviewed (1)
  • _scripts/package-lock.json: Generated file

Comment thread _scripts/package.json Outdated
Comment thread _scripts/package.json Outdated
Comment thread _scripts/package.json Outdated
Format _scripts/package.json and drop unused npm devDependency, keeping json only.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

@javi0guerrero4-cloud javi0guerrero4-cloud left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Update

@jakebailey jakebailey closed this Jul 6, 2026
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.

4 participants