Skip to content

chore: resolve all dependabot PRs, pin actions to latest SHA, add root docs - #81

Closed
BankkRoll wants to merge 2 commits into
alveusgg:mainfrom
BankkRoll:upstream-dep-updates
Closed

chore: resolve all dependabot PRs, pin actions to latest SHA, add root docs#81
BankkRoll wants to merge 2 commits into
alveusgg:mainfrom
BankkRoll:upstream-dep-updates

Conversation

@BankkRoll

@BankkRoll BankkRoll commented Jul 2, 2026

Copy link
Copy Markdown

Resolves all open dependabot PRs and adds missing root-level repo files. No new features — purely dependency hygiene and docs.

Resolves dependabot PRs: #73, #70, #68, #46, #45, #44, #43, #42, #41, #39, #37, #35


Dependency updates

npm packages

GitHub Actions (resolves #68, #73)

  • cloudflare/wrangler-action v3.15.0 → v4.0.0 (pinned SHA)
  • actions/checkout v6.0.2 → v7.0.0 (Node 16 → Node 20)
  • pnpm/action-setup v6.0.8 → v6.0.9
  • docker/setup-buildx-action v4.0.0 → v4.2.0
  • docker/login-action v4.1.0 → v4.3.0
  • docker/metadata-action v6.0.0 → v6.2.0
  • docker/build-push-action v7.1.0 → v7.3.0
  • pulumi/auth-actions v1.0.1 → v2.1.0
  • getsentry/action-release v3.6.0 → v3.7.0
  • All actions pinned to exact commit SHA

Code changes required by upgrades

drizzle-orm 0.36+ (part of the 0.33→0.45 range):

  1. pgTable() third-arg callback must return an array instead of an object — updated all 12 schema files in census/api/src/db/schema/
  2. drizzle(client, options) two-arg form replaced with drizzle({ client, ...options }) — updated census/api/src/db/db.ts

fastify v5: server.listen() no longer accepts a callback — converted to await server.listen() in census/api/src/index.ts


Docs

  • Added README.md at repo root (monorepo structure, tech stack, local setup, deployment overview)
  • Added CONTRIBUTING.md, SECURITY.md, LICENSE

BankkRoll added 2 commits July 2, 2026 09:23
- Bump vite 5.4.12 -> 5.4.21, react-router 6.26.1 -> 6.30.4, ipx 3.0.1 -> 3.1.1
- Bump fastify 4 -> 5, @fastify/cors 9 -> 10, @fastify/websocket 10 -> 11
- Bump drizzle-orm 0.33 -> 0.45, drizzle-kit 0.24 -> 0.31
- Bump @mux/mux-node 9.0.1 -> 9.1.0
- Migrate all pgTable() index callbacks from object to array syntax (drizzle 0.36+ requirement)
- Convert fastify listen() callback to async/await (fastify v5 requirement)
- Update wrangler-action v3 -> v4 in CI
- Pin all GitHub Actions to latest SHA: checkout v7, docker/* v4.2-v7.3, pnpm-setup v6.0.9, pulumi/auth-actions v2.1.0, sentry v3.7.0
- Add README.md, CONTRIBUTING.md, SECURITY.md, LICENSE to repo root

@MattIPv4 MattIPv4 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👀 Thanks for doing this, but I'm not keen on accepting dependency changes from external contributors. We have Dependabot configured to handle updating our dependencies.

I'd also ask that you keep PRs relatively atomic and focused on a single change -- adding docs should be done separately (and you're welcome to still do that) from other changes like changing dependencies.

@MattIPv4 MattIPv4 closed this Jul 2, 2026
@BankkRoll

Copy link
Copy Markdown
Author

👀 Thanks for doing this, but I'm not keen on accepting dependency changes from external contributors. We have Dependabot configured to handle updating our dependencies.

I'd also ask that you keep PRs relatively atomic and focused on a single change -- adding docs should be done separately (and you're welcome to still do that) from other changes like changing dependencies.

Thanks, I understand. You have many with CVE's so would get them updated then. feel free to check my github. Full-stack developer and Whitehat.

@MattIPv4

MattIPv4 commented Jul 2, 2026

Copy link
Copy Markdown
Member

👍 I'm sure we'll get the Dependabot updates merged in good time. I don't think any of the CVEs against the dependencies pose an actual risk to us, but if you discover something, please follow our existing security policy to report it.

@BankkRoll

BankkRoll commented Jul 2, 2026

Copy link
Copy Markdown
Author

No worries, yeah most are dev tooling and certs/signatures but no untrusted chains or RSA sigs being verified in the site. Will report if i ever come a crossed some thanks!

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