Skip to content

Release: staging -> main (2026-07-14 00:29)#1112

Merged
is0692vs merged 42 commits into
mainfrom
staging
Jul 14, 2026
Merged

Release: staging -> main (2026-07-14 00:29)#1112
is0692vs merged 42 commits into
mainfrom
staging

Conversation

@github-actions

@github-actions github-actions Bot commented Jul 14, 2026

Copy link
Copy Markdown

Summary

Promotes staging into main.

  • Source branch: staging
  • Target branch: main
  • Generated by npm run pr:promote

Included PRs

Compare

Greptile Summary

staging → main へのリリースPRです。アクセシビリティ改善(設定画面の保存メッセージへの条件付きARIAロール付与)と依存パッケージのアップデート(p-map v4→v7、@cloudflare/workers-types v4→v5)が主な変更内容です。

  • ARIA改善: 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

Filename Overview
.Jules/palette.md アクセシビリティ改善の知見(条件付きARIAロールパターン)をナレッジベースに追記したドキュメント更新のみ。コードへの影響なし。
apps/api/package.json p-map v4→v7(3メジャーバージョン飛び越え)および @cloudflare/workers-types v4→v5 のメジャーバージョンアップ。p-map v5以降はESM専用だがWrangler/esbuildがバンドル時に吸収するため実用上問題なし。
apps/web/src/app/settings/page.tsx 保存結果メッセージの <p> 要素にARIAロール(エラー時は alert、成功時は status)を条件付きで付与するアクセシビリティ改善。実装は正しく、既存の messageType ロジックと整合している。
package-lock.json package.json の変更内容(p-map v7、@cloudflare/workers-types v5)に対応したロックファイル更新。aggregate-errorclean-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
Loading
%%{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 --> B
Loading
Prompt To Fix All With AI
Fix the following 1 code review issue. Work through them one at a time, proposing concise fixes.

---

### Issue 1 of 1
apps/api/package.json:34
`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` の通過を確認しておくと安心です。

Reviews (1): Last reviewed commit: "Merge pull request #1090 from Hiroki-org..." | Re-trigger Greptile

Greptile also left 1 inline comment on this PR.

google-labs-jules Bot and others added 30 commits June 28, 2026 15:42
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
…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>
google-labs-jules Bot and others added 9 commits July 12, 2026 07:49
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
@github-actions github-actions Bot added automated Automated main-to-staging sync PR release Release promotion PR labels Jul 14, 2026
…p-7.0.5

chore(deps): bump p-map from 4.0.0 to 7.0.5
@vercel

vercel Bot commented Jul 14, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
open-shelf Ignored Ignored Jul 14, 2026 12:29am

…thub-actions-all-6d1c06d137

chore(deps): bump actions/cache from 5 to 6 in the github-actions-all group
@github-actions github-actions Bot changed the title Release: staging -> main (2026-07-14 00:27) Release: staging -> main (2026-07-14 00:28) Jul 14, 2026
…6728139762166

🧪 Add missing tests for normalizeOrigin URL parsing errors
@pull-request-size pull-request-size Bot added size/L and removed size/S labels Jul 14, 2026
@github-actions github-actions Bot changed the title Release: staging -> main (2026-07-14 00:28) Release: staging -> main (2026-07-14 00:29) Jul 14, 2026
@codecov

codecov Bot commented Jul 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

Comment thread apps/api/package.json
"drizzle-orm": "^0.45.2",
"hono": "^4.12.23",
"p-map": "^4.0.0"
"p-map": "^7.0.5"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P2 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!

@is0692vs
is0692vs merged commit e91b8d8 into main Jul 14, 2026
37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automated Automated main-to-staging sync PR release Release promotion PR size/L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant