Conversation
Co-authored-by: is0692vs <135803462+is0692vs@users.noreply.github.com>
Co-authored-by: is0692vs <135803462+is0692vs@users.noreply.github.com>
…710524347307 🔒 Prevent JWT secret leakage in token cache
…3596437525124151 🧪 [Testing] Add coverage for authHeaders SSR edge case
…iewer Co-authored-by: is0692vs <135803462+is0692vs@users.noreply.github.com>
Co-authored-by: is0692vs <135803462+is0692vs@users.noreply.github.com>
Co-authored-by: is0692vs <135803462+is0692vs@users.noreply.github.com>
…iewer Co-authored-by: is0692vs <135803462+is0692vs@users.noreply.github.com>
…92300292320040 🛡️ Sentinel: [セキュリティ強化] 脆弱な乱数生成器の安全なUUIDへの置き換え
# Conflicts: # apps/web/src/components/toast.tsx
…403905686049' into codex-openshelf-1057
…6258403905686049 🎨 Palette: アクセシビリティとUXの改善 (Toast, Tag Autocomplete, PDF Viewer)
Co-authored-by: is0692vs <135803462+is0692vs@users.noreply.github.com>
…509e6 🧪 Add tests for UserPageClient component
…signing aria roles Co-authored-by: is0692vs <135803462+is0692vs@users.noreply.github.com>
…66046188642 🎨 Palette: トースト通知のアクセシビリティ改善(動的role属性の追加)
Added `role={messageType === 'error' ? 'alert' : 'status'}` to the dynamic message container in the settings page to improve screen reader feedback.
Co-authored-by: is0692vs <135803462+is0692vs@users.noreply.github.com>
Co-authored-by: is0692vs <135803462+is0692vs@users.noreply.github.com>
Added `role={messageType === 'error' ? 'alert' : 'status'}` to the dynamic message container in the settings page to improve screen reader feedback.
Co-authored-by: is0692vs <135803462+is0692vs@users.noreply.github.com>
…-roles-10022751178647911450 🎨 Palette: 保存結果メッセージに適切なARIAロールを追加
Bumps the github-actions-all group with 1 update: [actions/cache](https://github.com/actions/cache). Updates `actions/cache` from 5 to 6 - [Release notes](https://github.com/actions/cache/releases) - [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md) - [Commits](actions/cache@v5...v6) --- updated-dependencies: - dependency-name: actions/cache dependency-version: '6' dependency-type: direct:production update-type: version-update:semver-major dependency-group: github-actions-all ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [p-map](https://github.com/sindresorhus/p-map) from 4.0.0 to 7.0.5. - [Release notes](https://github.com/sindresorhus/p-map/releases) - [Commits](sindresorhus/p-map@v4.0.0...v7.0.5) --- updated-dependencies: - dependency-name: p-map dependency-version: 7.0.5 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [@cloudflare/workers-types](https://github.com/cloudflare/workerd) from 4.20260624.1 to 5.20260713.1. - [Release notes](https://github.com/cloudflare/workerd/releases) - [Changelog](https://github.com/cloudflare/workerd/blob/main/RELEASE.md) - [Commits](https://github.com/cloudflare/workerd/commits) --- updated-dependencies: - dependency-name: "@cloudflare/workers-types" dependency-version: 5.20260713.1 dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
…dflare/workers-types-5.20260713.1 chore(deps-dev): bump @cloudflare/workers-types from 4.20260624.1 to 5.20260713.1
…p-7.0.5 chore(deps): bump p-map from 4.0.0 to 7.0.5
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
…thub-actions-all-6d1c06d137 chore(deps): bump actions/cache from 5 to 6 in the github-actions-all group
…6728139762166 🧪 Add missing tests for normalizeOrigin URL parsing errors
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
| "drizzle-orm": "^0.45.2", | ||
| "hono": "^4.12.23", | ||
| "p-map": "^4.0.0" | ||
| "p-map": "^7.0.5" |
There was a problem hiding this comment.
p-map を v4 から v7 へ3メジャーバージョン一気に上げています。v5以降は ESM専用 となり、CommonJS の require() では読み込めません(ERR_REQUIRE_ESM)。apps/api/package.json の "type": "commonjs" との組み合わせが気になりますが、実際の実行はWrangler/esbuildがバンドルするため本番デプロイ・テスト(vitest)ともに問題ないことを確認しています。ただし、pMap 呼び出し箇所(papers.ts L134, L806)はいずれもマッパー内でエラーをキャッチしているため、v5で変わった AggregateError の挙動にも影響を受けません。stagingで動作確認済みであれば問題なしと判断しますが、念のため wrangler dev および vitest の通過を確認しておくと安心です。
Prompt To Fix With AI
This is a comment left during a code review.
Path: apps/api/package.json
Line: 34
Comment:
`p-map` を v4 から v7 へ3メジャーバージョン一気に上げています。v5以降は **ESM専用** となり、CommonJS の `require()` では読み込めません(`ERR_REQUIRE_ESM`)。`apps/api/package.json` の `"type": "commonjs"` との組み合わせが気になりますが、実際の実行はWrangler/esbuildがバンドルするため本番デプロイ・テスト(vitest)ともに問題ないことを確認しています。ただし、`pMap` 呼び出し箇所(papers.ts L134, L806)はいずれもマッパー内でエラーをキャッチしているため、v5で変わった `AggregateError` の挙動にも影響を受けません。stagingで動作確認済みであれば問題なしと判断しますが、念のため `wrangler dev` および `vitest` の通過を確認しておくと安心です。
How can I resolve this? If you propose a fix, please make it concise.Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
Summary
Promotes
stagingintomain.stagingmainnpm run pr:promoteIncluded PRs
Compare
Greptile Summary
staging → main へのリリースPRです。アクセシビリティ改善(設定画面の保存メッセージへの条件付きARIAロール付与)と依存パッケージのアップデート(
p-mapv4→v7、@cloudflare/workers-typesv4→v5)が主な変更内容です。settings/page.tsxの保存結果メッセージにrole={messageType === "error" ? "alert" : "status"}を追加。エラーはスクリーンリーダーへ即座に、成功メッセージは丁寧に通知される正しい実装。p-mapが3メジャーバージョン一気に上がり v5以降はESM専用となるが、Wrangler/esbuildによるバンドルが吸収するため実用上の問題はなし。aggregate-errorなど不要になった推移的依存がlock fileからクリーンに削除されている。.Jules/palette.mdに条件付きARIAロールパターンの知見を追記。Confidence Score: 4/5
stagingで検証済みの変更をmainへ昇格するPRであり、全体的に安全です。
p-mapのv4→v7という大きなメジャーバージョンアップが含まれており、v5以降のESM専用化とpackage typeのcommonjs宣言の組み合わせは一見懸念されますが、実際の呼び出し箇所はいずれもマッパー内でエラーをキャッチしており、Wrangler/esbuildのバンドルがESM interopを処理するため実害はありません。ARIA改善は仕様に沿った正しい実装です。stagingでの動作確認が前提であれば概ね安心してマージできます。apps/api/package.jsonのp-mapメジャーバージョン変更。他のファイルは特に問題なし。Important Files Changed
p-mapv4→v7(3メジャーバージョン飛び越え)および@cloudflare/workers-typesv4→v5 のメジャーバージョンアップ。p-map v5以降はESM専用だがWrangler/esbuildがバンドル時に吸収するため実用上問題なし。<p>要素にARIAロール(エラー時はalert、成功時はstatus)を条件付きで付与するアクセシビリティ改善。実装は正しく、既存のmessageTypeロジックと整合している。aggregate-errorやclean-stackなどの推移的依存が削除されておりクリーン。Flowchart
%%{init: {'theme': 'neutral'}}%% flowchart TD A[staging ブランチ] -->|PR #1112| B[main ブランチ] subgraph 含まれるPR変更 C["#1111 @cloudflare/workers-types v4→v5\n p-map v4→v7"] D["#1086 / #1080 / #1057\n アクセシビリティ改善\n (ARIA role, Toast, Tag Autocomplete)"] E["#1070 / #1035\n テスト追加\n (UserPageClient, authHeaders SSR)"] F["#1058 / #1041\n セキュリティ強化\n (UUID乱数, JWTシークレット)"] end subgraph settings/page.tsx の変更 G["message && …"] --> H{messageType} H -->|error| I["role='alert'\n(スクリーンリーダーへ即座に通知)"] H -->|その他| J["role='status'\n(スクリーンリーダーへ丁寧に通知)"] end C & D & E & F --> B%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%% flowchart TD A[staging ブランチ] -->|PR #1112| B[main ブランチ] subgraph 含まれるPR変更 C["#1111 @cloudflare/workers-types v4→v5\n p-map v4→v7"] D["#1086 / #1080 / #1057\n アクセシビリティ改善\n (ARIA role, Toast, Tag Autocomplete)"] E["#1070 / #1035\n テスト追加\n (UserPageClient, authHeaders SSR)"] F["#1058 / #1041\n セキュリティ強化\n (UUID乱数, JWTシークレット)"] end subgraph settings/page.tsx の変更 G["message && …"] --> H{messageType} H -->|error| I["role='alert'\n(スクリーンリーダーへ即座に通知)"] H -->|その他| J["role='status'\n(スクリーンリーダーへ丁寧に通知)"] end C & D & E & F --> BPrompt To Fix All With AI
Reviews (1): Last reviewed commit: "Merge pull request #1090 from Hiroki-org..." | Re-trigger Greptile