Skip to content

chore: update dependencies#50

Merged
rubenhensen merged 2 commits into
mainfrom
chore/dep-updates-2026-05-24
May 25, 2026
Merged

chore: update dependencies#50
rubenhensen merged 2 commits into
mainfrom
chore/dep-updates-2026-05-24

Conversation

@dobby-coder
Copy link
Copy Markdown
Contributor

@dobby-coder dobby-coder Bot commented May 24, 2026

Summary

Patch/minor dep bumps surfaced by the dep-update scan:

  • pg-node@e4a/pg-js 1.9.0 → 1.10.0
  • pg-manualwebpack 5.107.0 → 5.107.1
  • pg-sveltekit@e4a/pg-js 1.8.0 → 1.10.0, @sveltejs/kit 2.60.1 → 2.61.1, prettier-plugin-svelte 4.0.0 → 4.0.1, vite 8.0.13 → 8.0.14

pg-dotnet has no outdated deps. No security advisories, no major bumps. All listed packages were confirmed to be imported by source files (not just manifest entries).

No CI workflow in this repo; please review manually.

Verification

  • pg-nodenpm install clean, syntax-check OK on all .mjs, import smoke-test of @e4a/pg-js succeeds.
  • pg-manualnpm install && npm run build succeeds with zero "export … was not found" warnings, webpack 5.107.1 in build banner.
  • pg-sveltekitnpm install && npm run build && npm run check, all clean (0 svelte-check errors/warnings); build.target: 'esnext' retained in vite.config.ts.

No test suite in any sub-project — no tests to verify.

Closes #49

- pg-node: @e4a/pg-js 1.9.0 -> 1.10.0
- pg-manual: webpack 5.107.0 -> 5.107.1
- pg-sveltekit: @e4a/pg-js 1.8.0 -> 1.10.0, @sveltejs/kit 2.60.1 -> 2.61.1, prettier-plugin-svelte 4.0.0 -> 4.0.1, vite 8.0.13 -> 8.0.14

All bumps are patch/minor; no advisories. Build verified in each sub-project (and svelte-check clean for pg-sveltekit).
@dobby-coder dobby-coder Bot requested a review from rubenhensen May 24, 2026 22:15
@dobby-coder
Copy link
Copy Markdown
Contributor Author

dobby-coder Bot commented May 24, 2026

Rule-compliance sweep applied two fixes:

  1. pg-node/README.md version drift (commit 7582efd) — the line "package.json depends on the published @e4a/pg-js (currently ^1.9.0)" was pointing at the pre-bump version. Updated to ^1.10.0 to match the new package.json. Caught by docs-drift-check.

  2. PR body manual-review callout — added "No CI workflow in this repo; please review manually." per no-ci-repos-manual-review (postguard-examples is listed there as a known no-CI repo, so the auto-flip-after-CI flow doesn't apply and reviewers need an explicit nudge).

Copy link
Copy Markdown
Contributor Author

@dobby-coder dobby-coder Bot left a comment

Choose a reason for hiding this comment

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

Bot can't --approve its own PR; posting as comment instead. Verdict: LGTM, ready to merge once a human approves.

Verified locally — patch/minor bumps only, no security advisories, no major-version risk.

  • pg-nodenpm install clean, node --check passes on all .mjs, @e4a/pg-js@1.10.0 imports and exposes the expected surface (PostGuard, Sealed, Opened, error types, RecipientBuilder, PG_MAX_*).
  • pg-manualnpm run build succeeds with zero "export … was not found" warnings, banner reads webpack 5.107.1.
  • pg-sveltekitnpm run build succeeds, npm run check clean (0 errors / 0 warnings / 0 files-with-problems), installed @e4a/pg-js resolves to 1.10.0.

All bumped packages are imported from real source (not dead code): pg-node/src/encryption.mjs, pg-sveltekit/src/lib/postguard/encryption.ts, pg-manual/webpack.config.js. README version-reference fix in commit 7582efd matches the new package.json constraint.

@dobby-coder
Copy link
Copy Markdown
Contributor Author

dobby-coder Bot commented May 25, 2026

FIX stage triggered, but no human review feedback is present on this PR — only this agent's own self-comment and self-review. Nothing to address; PR is unchanged. Re-requesting maintainer review is skipped to avoid re-pinging when nothing has changed since the original open.

Copy link
Copy Markdown
Contributor Author

@dobby-coder dobby-coder Bot left a comment

Choose a reason for hiding this comment

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

Cycle 1 review — re-verified locally. Verdict: LGTM, ready to merge once a human approves. (GitHub blocks self-approval since this PR was opened by the same bot identity.)

  • pg-node: npm install clean; @e4a/pg-js@1.10.0 imports OK, expected API surface present.
  • pg-manual: npm run build succeeds with webpack 5.107.1.
  • pg-sveltekit: npm run build + npm run check clean (0 errors / 0 warnings); resolved versions match the bumps (@e4a/pg-js@1.10.0, @sveltejs/kit@2.61.1, prettier-plugin-svelte@4.0.1, vite@8.0.14).
  • README version reference in pg-node/README.md matches the new ^1.10.0 constraint.

Patch/minor only, no advisories, no breaking changes. Conventional Commit check green.

Copy link
Copy Markdown
Contributor Author

@dobby-coder dobby-coder Bot left a comment

Choose a reason for hiding this comment

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

Verdict: looks good to merge (GitHub blocks the bot from formally approving its own PR, so leaving this as a comment review).

Verified locally on the PR branch:

  • pg-manualnpm install && npm run build succeeds; banner shows webpack 5.107.1 and compiled successfully.
  • pg-nodenpm install clean; node --check on index.mjs and src/encryption.mjs passes.
  • pg-sveltekitnpm install && npm run build && npm run check all green (0 ERRORS 0 WARNINGS); @e4a/pg-js resolves to 1.10.0 pulling the updated pg-wasm 0.6.1 and yivi 1.x deps.

All bumped packages are imported by source (@e4a/pg-js in encryption.mjs / encryption.ts, webpack in pg-manual/webpack.config.js) — not lockfile-only. README version drift is fixed (^1.10.0). PR closes #49 via a closing keyword and uses a conventional title. No CI workflow in this repo, so the merge is yours when ready.

Copy link
Copy Markdown
Contributor Author

@dobby-coder dobby-coder Bot left a comment

Choose a reason for hiding this comment

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

Verdict: LGTM / ready to merge. (GitHub blocks --approve because the PR author and this reviewer are the same bot identity.)

Reviewed and verified locally on chore/dep-updates-2026-05-24.

Verification (matches PR body)

  • pg-nodenpm install clean (0 vulns), all .mjs node --check pass, import('@e4a/pg-js') resolves and exposes the expected keys (DecryptionError, Opened, PG_MAX_ATTACHMENT_SIZE, …). pg-node/README.md line 30 correctly references ^1.10.0.
  • pg-manualnpm run build succeeds with banner examples (webpack 5.107.1) compiled successfully in 276 ms, no missing-export warnings.
  • pg-sveltekitnpm install reports 0 vulnerabilities, npm run build succeeds, npm run check reports 0 ERRORS 0 WARNINGS 0 FILES_WITH_PROBLEMS over 294 files, vite.config.ts retains build.target: 'esnext'.

Usage check

  • @e4a/pg-js is imported by pg-node/src/encryption.mjs and pg-sveltekit/src/lib/postguard/encryption.ts.
  • webpack drives pg-manual/webpack.config.js.

All bumps are patch/minor, scope matches the issue, closes #49.

Note (not blocking, not introduced by this PR): npm audit in pg-manual now reports 5 pre-existing moderate advisories in dev deps (qs, uuid via sockjs/webpack-dev-server). Worth tracking separately if you want them cleaned up.

Copy link
Copy Markdown
Contributor Author

@dobby-coder dobby-coder Bot left a comment

Choose a reason for hiding this comment

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

Cycle 2 — head 7582efd unchanged since prior reviews; no human feedback to address. Verified locally: pg-node import smoke OK, pg-manual builds with webpack 5.107.1, pg-sveltekit build + check clean (0 errors / 0 warnings). Verdict: LGTM, ready to merge. GitHub blocks self-approval; flagging as approved upstream to the pipeline.

@rubenhensen rubenhensen merged commit 8f312b9 into main May 25, 2026
2 checks passed
@rubenhensen rubenhensen deleted the chore/dep-updates-2026-05-24 branch May 25, 2026 12:29
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.

chore: update dependencies

1 participant