Repository navigation
chore: Release packages - #54
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (13)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe changes update asynchronous test callbacks and image decode mocks, run initial picture-source assertions concurrently, update four package versions and changelogs to 0.3.1, and process package changelogs concurrently. ChangesPromise handling in tests
0.3.1 release metadata
Priority: ➖ Normal Estimated code review effort: 2 (Simple) | ~10 minutes Change: Other Merge Risk: ⚪ Minimal · up to No concrete test-helper or release-metadata regression is established, so the change appears mergeable subject to normal checks. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 8 functions across 5 files. (8 skipped: 8 unsupported.)
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. A rabbit checks each promise returned, Comment |
|
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
Version bumps are consistent across all packages, the changelog dependency claims match the lockfile, no runtime source changed, and the test/script refactors are semantically equivalent lint cleanups.
Review effort: Balanced
Findings: None
What changed in this PR
This is a changesets-driven release PR that bumps the four fixed-version packages (@desource/image, @desource/image-angular, @desource/image-react, @desource/image-svelte) from 0.3.0 to 0.3.1 and records the corresponding Patch Changes in each CHANGELOG.md. The release captures a round of dependency upgrades (std-env 4.3.0, Vite 8.3.1, Vitest + V8 coverage 5.0.2, pnpm 12.8.1). Alongside the release artifacts, it includes small require-await-style refactors in test helpers and the changelog-formatting script — i.e. async arrow functions that never await were rewritten to return Promise.resolve(...)/Promise.reject(...) or to use Promise.all, which is consistent with the linting/tooling upgrades mentioned in the changelog.
Changes:
- Bumped all four packages to
0.3.1and added matching changelog entries (core omits an internal dependency line since it is@desource/imageitself). - Refactored test setup/helpers (
setup.tsx,mock-image.ts,placeholder-lifecycle.ts,DsPicture.ts) to drop no-opasyncwrappers — semantically equivalent. - Refactored
format-changelog-dependencies.mtsfrom a sequential loop toPromise.all, preserving the formatted-file count.
| File | Description |
|---|---|
| packages/core/package.json | Version bump 0.3.0 → 0.3.1 |
| packages/core/CHANGELOG.md | Adds 0.3.1 patch-changes entry (no self internal-dep line) |
| packages/react/package.json | Version bump 0.3.0 → 0.3.1 |
| packages/react/CHANGELOG.md | Adds 0.3.1 entry incl. @desource/image@0.3.1 dep update |
| packages/svelte/package.json | Version bump 0.3.0 → 0.3.1 |
| packages/svelte/CHANGELOG.md | Adds 0.3.1 entry incl. internal dep update |
| packages/angular/package.json | Version bump 0.3.0 → 0.3.1 |
| packages/angular/CHANGELOG.md | Adds 0.3.1 entry incl. internal dep update |
| scripts/format-changelog-dependencies.mts | Sequential loop → Promise.all; count preserved |
| packages/react/test/unit/setup.tsx | Drops no-op async wrappers around act() |
| common/test/unit/setup/mock-image.ts | async () => undefined → () => Promise.resolve() |
| common/test/unit/placeholder-lifecycle.ts | Same equivalent Promise refactors in tests |
| common/test/e2e/DsPicture.ts | Sequential attribute checks → Promise.all over distinct locators |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.



Summary by CodeRabbit