Skip to content

chore: pin coder to v2.33.1 and bump deps#941

Merged
EhabY merged 2 commits intomainfrom
chore/pin-coder-v2.33.1-and-deps
May 7, 2026
Merged

chore: pin coder to v2.33.1 and bump deps#941
EhabY merged 2 commits intomainfrom
chore/pin-coder-v2.33.1-and-deps

Conversation

@EhabY
Copy link
Copy Markdown
Collaborator

@EhabY EhabY commented May 7, 2026

  • Pin the vendored coder client to github:coder/coder#v2.33.1 (previously #main) so installs are reproducible and not re-fetched on every pnpm install.
  • Update CONTRIBUTING.md with the new tag-pinning convention.
  • Bump misc deps to latest minors/patches: axios, zod, eslint, typescript-eslint, jsdom, globals, @eslint-react/eslint-plugin, @tanstack/react-query, react, react-dom, vite.

@EhabY EhabY force-pushed the chore/pin-coder-v2.33.1-and-deps branch 2 times, most recently from cb2f8fe to 9d7fd85 Compare May 7, 2026 10:21
EhabY added 2 commits May 7, 2026 13:24
- Pin coder to github:coder/coder#v2.33.1 (was #main) for reproducible
  builds; document the tagging convention in CONTRIBUTING.md.
- Bump axios, zod, eslint, typescript-eslint, jsdom, globals,
  @eslint-react/eslint-plugin, @tanstack/react-query, react,
  react-dom, vite to latest minors/patches.
- Cast "agents" to Experiment in deploymentManager: the experiment
  string is not in the v2.33.1 Experiment union (added in v2.34.0-rc.0).
  Runtime behavior is unchanged.
web-api-no-leaked-fetch and jsx-no-leaked-dollar ship at warn level in
v5's recommended-type-checked preset. They catch real bug classes
(unaborted fetch in useEffect; accidentally-rendered template strings).
Existing code passes both, so promote to error.
@EhabY EhabY force-pushed the chore/pin-coder-v2.33.1-and-deps branch from 9d7fd85 to e1e36e7 Compare May 7, 2026 10:24
@EhabY EhabY self-assigned this May 7, 2026
Comment thread src/deployment/deploymentManager.ts
@EhabY EhabY merged commit 106516c into main May 7, 2026
10 checks passed
@EhabY EhabY deleted the chore/pin-coder-v2.33.1-and-deps branch May 7, 2026 15:07
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.

2 participants