Skip to content

Release: staging -> main (2026-07-22 04:26)#1148

Open
github-actions[bot] wants to merge 32 commits into
mainfrom
staging
Open

Release: staging -> main (2026-07-22 04:26)#1148
github-actions[bot] wants to merge 32 commits into
mainfrom
staging

Conversation

@github-actions

@github-actions github-actions Bot commented Jul 22, 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

このPRは、キーボード操作時のフォーカス表示を複数のUI部品に追加します。

  • Header のリンクと認証ボタンにフォーカスリングを追加。
  • FeedButton の起動、コピー、外部リンク操作にフォーカスリングを追加。
  • MarkdownEditor のタブ操作にフォーカスリングを追加。
  • PdfViewer のツールバー、検索、エラー時ダウンロード操作にフォーカスリングを追加。
  • フォーカスリング方針を .Jules/palette.md に追記。

Confidence Score: 5/5

This looks safe to merge after a small focus-style consistency cleanup.

  • No blocking issues found in the changed code.
  • The remaining note is limited to inconsistent focus styling between two PDF fallback download paths.

apps/web/src/components/pdf-viewer.tsx

Important Files Changed

Filename Overview
.Jules/palette.md キーボードフォーカス可視性に関する方針メモが追加されています。
apps/web/src/components/feed-button.tsx FeedButton の起動、コピー、外部リンク操作にフォーカスリングが追加されています。
apps/web/src/components/header.tsx ヘッダー内のリンクとログイン/ログアウトボタンにフォーカスリングが追加されています。
apps/web/src/components/markdown-editor.tsx MarkdownEditor の Write/Preview タブにフォーカスリングが追加されています。
apps/web/src/components/pdf-viewer.tsx PdfViewer の各操作ボタンと一部のエラー時ダウンロード操作にフォーカスリングが追加されています。
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/web/src/components/pdf-viewer.tsx:617
**Fallback Focus Styles Diverge**

When `onDownloadFallback` is provided, the download fallback gets the new custom focus ring, but the sibling `<a>` fallback for the same action still only uses `underline`. In the PDF load-error state, keyboard users get different focus indicators for the same download action depending on which prop shape reaches `PdfViewer`, so the accessibility behavior is inconsistent.

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

Greptile also left 1 inline comment on this PR.

Context used:

Copilot AI and others added 30 commits July 18, 2026 05:32
chore: sync main into staging
…rror

Restore sanitized invite-route error logging and tighten regression assertions
Sanitize invitee email lookup error logging in papers invites route
…character-counter

Guard character-counter accessibility contract (no live regions) and align palette guidance
Co-authored-by: is0692vs <135803462+is0692vs@users.noreply.github.com>
Bumps the github-actions-all group with 2 updates: [actions/checkout](https://github.com/actions/checkout) and [actions/setup-node](https://github.com/actions/setup-node).


Updates `actions/checkout` from 7.0.0 to 7.0.1
- [Release notes](https://github.com/actions/checkout/releases)
- [Commits](actions/checkout@v7...v7.0.1)

Updates `actions/setup-node` from 6 to 7
- [Release notes](https://github.com/actions/setup-node/releases)
- [Commits](actions/setup-node@v6...v7)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-version: 7.0.1
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: github-actions-all
- dependency-name: actions/setup-node
  dependency-version: '7'
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: github-actions-all
...

Signed-off-by: dependabot[bot] <support@github.com>
…dates

Bumps the npm-development-minor-patch group with 7 updates:

| Package | From | To |
| --- | --- | --- |
| [@typescript-eslint/eslint-plugin](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/eslint-plugin) | `8.64.0` | `8.65.0` |
| [@typescript-eslint/parser](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/parser) | `8.64.0` | `8.65.0` |
| [postcss](https://github.com/postcss/postcss) | `8.5.19` | `8.5.20` |
| [@cloudflare/workers-types](https://github.com/cloudflare/workerd) | `5.20260713.1` | `5.20260719.1` |
| [wrangler](https://github.com/cloudflare/workers-sdk/tree/HEAD/packages/wrangler) | `4.110.0` | `4.112.0` |
| [@tailwindcss/postcss](https://github.com/tailwindlabs/tailwindcss/tree/HEAD/packages/@tailwindcss-postcss) | `4.3.2` | `4.3.3` |
| [tailwindcss](https://github.com/tailwindlabs/tailwindcss/tree/HEAD/packages/tailwindcss) | `4.3.2` | `4.3.3` |


Updates `@typescript-eslint/eslint-plugin` from 8.64.0 to 8.65.0
- [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases)
- [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/CHANGELOG.md)
- [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v8.65.0/packages/eslint-plugin)

Updates `@typescript-eslint/parser` from 8.64.0 to 8.65.0
- [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases)
- [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/parser/CHANGELOG.md)
- [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v8.65.0/packages/parser)

Updates `postcss` from 8.5.19 to 8.5.20
- [Release notes](https://github.com/postcss/postcss/releases)
- [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md)
- [Commits](postcss/postcss@8.5.19...8.5.20)

Updates `@cloudflare/workers-types` from 5.20260713.1 to 5.20260719.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)

Updates `wrangler` from 4.110.0 to 4.112.0
- [Release notes](https://github.com/cloudflare/workers-sdk/releases)
- [Commits](https://github.com/cloudflare/workers-sdk/commits/wrangler@4.112.0/packages/wrangler)

Updates `@tailwindcss/postcss` from 4.3.2 to 4.3.3
- [Release notes](https://github.com/tailwindlabs/tailwindcss/releases)
- [Changelog](https://github.com/tailwindlabs/tailwindcss/blob/main/CHANGELOG.md)
- [Commits](https://github.com/tailwindlabs/tailwindcss/commits/v4.3.3/packages/@tailwindcss-postcss)

Updates `tailwindcss` from 4.3.2 to 4.3.3
- [Release notes](https://github.com/tailwindlabs/tailwindcss/releases)
- [Changelog](https://github.com/tailwindlabs/tailwindcss/blob/main/CHANGELOG.md)
- [Commits](https://github.com/tailwindlabs/tailwindcss/commits/v4.3.3/packages/tailwindcss)

---
updated-dependencies:
- dependency-name: "@typescript-eslint/eslint-plugin"
  dependency-version: 8.65.0
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: npm-development-minor-patch
- dependency-name: "@typescript-eslint/parser"
  dependency-version: 8.65.0
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: npm-development-minor-patch
- dependency-name: postcss
  dependency-version: 8.5.20
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: npm-development-minor-patch
- dependency-name: "@cloudflare/workers-types"
  dependency-version: 5.20260719.1
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: npm-development-minor-patch
- dependency-name: wrangler
  dependency-version: 4.112.0
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: npm-development-minor-patch
- dependency-name: "@tailwindcss/postcss"
  dependency-version: 4.3.3
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: npm-development-minor-patch
- dependency-name: tailwindcss
  dependency-version: 4.3.3
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: npm-development-minor-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [@testing-library/jest-dom](https://github.com/testing-library/jest-dom) from 6.9.1 to 7.0.0.
- [Release notes](https://github.com/testing-library/jest-dom/releases)
- [Changelog](https://github.com/testing-library/jest-dom/blob/main/CHANGELOG.md)
- [Commits](testing-library/jest-dom@v6.9.1...v7.0.0)

---
updated-dependencies:
- dependency-name: "@testing-library/jest-dom"
  dependency-version: 7.0.0
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
…thub-actions-all-3d4a5765df

chore(deps): bump the github-actions-all group with 2 updates
…development-minor-patch-43847a5e92

chore(deps-dev): bump the npm-development-minor-patch group with 7 updates
# Conflicts:
#	apps/web/package.json
#	package-lock.json
…ing-library/jest-dom-7.0.0

chore(deps-dev): bump @testing-library/jest-dom from 6.9.1 to 7.0.0
Co-authored-by: is0692vs <135803462+is0692vs@users.noreply.github.com>
…ck-14096522148886484374

🧹 Extract setupOrgApiMock logic into smaller helper functions
google-labs-jules Bot and others added 2 commits July 22, 2026 04:10
Co-authored-by: is0692vs <135803462+is0692vs@users.noreply.github.com>
…2876612747669

🎨 Palette: キーボードナビゲーションのフォーカスリングを追加
@github-actions github-actions Bot added automated Automated main-to-staging sync PR release Release promotion PR labels Jul 22, 2026
@vercel

vercel Bot commented Jul 22, 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 22, 2026 4:26am

<button
type="button"
className="underline"
className="underline focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-offset-2 focus-visible:ring-gray-900 dark:focus-visible:ring-gray-100 dark:focus-visible:ring-offset-gray-950 rounded-sm"

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 Fallback Focus Styles Diverge

When onDownloadFallback is provided, the download fallback gets the new custom focus ring, but the sibling <a> fallback for the same action still only uses underline. In the PDF load-error state, keyboard users get different focus indicators for the same download action depending on which prop shape reaches PdfViewer, so the accessibility behavior is inconsistent.

Knowledge Base Used: Paper Upload, Storage, and Viewing Flow

Prompt To Fix With AI
This is a comment left during a code review.
Path: apps/web/src/components/pdf-viewer.tsx
Line: 617

Comment:
**Fallback Focus Styles Diverge**

When `onDownloadFallback` is provided, the download fallback gets the new custom focus ring, but the sibling `<a>` fallback for the same action still only uses `underline`. In the PDF load-error state, keyboard users get different focus indicators for the same download action depending on which prop shape reaches `PdfViewer`, so the accessibility behavior is inconsistent.

**Knowledge Base Used:** [Paper Upload, Storage, and Viewing Flow](https://app.greptile.com/hiroki-org/-/custom-context/knowledge-base/hiroki-org/openshelf/-/docs/papers-file-flow.md)

How can I resolve this? If you propose a fix, please make it concise.

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/M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants