chore: update dependencies and devDependencies in package.json #465
+1,720
−2,852
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Upgraded probot to version 14.0.2. - Updated @types/node to version 24.6.2. - Added new devDependencies: @vitest/coverage-v8, @vitest/ui, @vitest/utils, vite. - Updated nock to version 14.0.10 and smee-client to version 4.3.1. - Updated typescript to version 5.9.3 and vitest to version 3.2.4. fix: add skipLibCheck to tsconfig.json for improved type checking performance feat: add 'smee' to workspace settings in openci.code-workspace
Walkthrough
Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
Disabled knowledge base sources:
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
⛔ Files ignored due to path filters (1)
openci-runner/github-apps/package-lock.jsonis excluded by!**/package-lock.json
📒 Files selected for processing (3)
openci-runner/github-apps/package.json(1 hunks)openci-runner/github-apps/tsconfig.json(1 hunks)openci.code-workspace(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: deploy-firebase-functions-dev
🔇 Additional comments (2)
openci.code-workspace (1)
80-80: LGTM!
smee-clientの依存関係更新に合わせて、スペルチェック辞書に"smee"を追加しています。変更は適切です。openci-runner/github-apps/package.json (1)
19-27: 依存関係のバージョンはすべてnpm上に存在し、既知の脆弱性も確認されませんでした。
| }, | ||
| "dependencies": { | ||
| "probot": "^13.4.5" | ||
| "probot": "^14.0.2" |
There was a problem hiding this comment.
🧩 Analysis chain
Probot 14.0.2への破壊的変更を確認してください。
Probotをv13からv14へのメジャーバージョンアップグレードしています。破壊的変更が含まれている可能性が高いため、以下を確認してください：
- APIの変更による既存コードへの影響
- 設定ファイルの変更要件
- 非推奨機能の削除
以下のスクリプトでProbotのリリースノートと破壊的変更を確認できます：
🏁 Script executed:
#!/bin/bash
# Probot v14の破壊的変更とリリースノートを確認
# GitHubのリリースノートを取得
gh api repos/probot/probot/releases --jq '.[] | select(.tag_name | startswith("v14")) | {tag: .tag_name, name: .name, body: .body}' | head -50
# v13とv14の間の主な変更を検索
gh api graphql -f query='
{
repository(owner: "probot", name: "probot") {
releases(first: 20, orderBy: {field: CREATED_AT, direction: DESC}) {
nodes {
tagName
name
description
createdAt
}
}
}
}'Length of output: 28458
メジャーバージョン14への移行対応を実施してください。
- ESM専用化: package.jsonに
"type": "module"を追加し、CommonJSのrequire()をimportに置換 - TypeScript設定を更新:
"module":"node16"/"moduleResolution":"node16"（またはTS5.9+で"node20"）に変更 - Webhook型定義を
@octokit/openapi-webhooks-types-migrationからインポート - REST API呼び出しを
context.octokit.rest.*に変更 - 内蔵Expressサーバーが削除されたため、必要に応じて
expressをインストールし独自サーバー設定へ移行 createNodeMiddleware()をawait付き非同期呼び出しに更新- デフォルトのリスニングホストが
localhostになったため、必要時はHOST環境変数または--hostフラグを指定 - 必要に応じて
@sentry/nodeとioredisを個別インストール - Node.jsの最低サポートバージョンを
20.18+または22+に合わせ、enginesフィールドを確認
🤖 Prompt for AI Agents
In openci-runner/github-apps/package.json around line 16, update the project to
be compatible with Probot v14: add "type": "module" to package.json and convert
all CommonJS require() usages to ES module imports; update tsconfig to use
"module":"node16" and "moduleResolution":"node16" (or "node20" if using TS
5.9+); switch webhook types to import from
@octokit/openapi-webhooks-types-migration; change REST calls to use
context.octokit.rest.*; install and configure express if you depended on the
built-in Express server and migrate server code accordingly; make
createNodeMiddleware() an awaited async call; ensure the default host behavior
is handled via HOST env or --host flag where needed; add @sentry/node and
ioredis to dependencies if used; and verify engines field requires Node 20.18+
or 22+.
Comment on lines
+18
to
+20
| "forceConsistentCasingInFileNames": true, | ||
| // TODO(someone): remove this | ||
| "skipLibCheck": true, |
There was a problem hiding this comment.
TODOコメントを明確化し、技術的負債として追跡してください。
skipLibCheck: trueは依存関係の型チェックをスキップするため、ビルド時間は改善されますが、依存関係の型エラーを隠蔽する可能性があります。大規模な依存関係更新時の一時的な回避策としては妥当ですが、TODOコメントが曖昧です。
以下の対応を推奨します：
- TODOに担当者と期限を追加（例：
TODO(mafreud, 2025-10-31): remove this after verifying all type definitions） - または、この技術的負債を追跡するためのissueを作成
この技術的負債を追跡するissueの作成をお手伝いしましょうか？
🤖 Prompt for AI Agents
In openci-runner/github-apps/tsconfig.json around lines 18 to 20, the TODO
comment for "skipLibCheck": true is ambiguous—update the TODO to clearly track
this technical debt by adding an owner and a deadline (e.g., TODO(mafreud,
2025-10-31): remove this after verifying all type definitions) or replace the
inline TODO with a reference to a created issue (e.g., TODO: see ISSUE-1234) and
then create that issue describing why skipLibCheck was added, the verification
steps, and the planned removal timeframe so the debt is tracked and actionable.
There was a problem hiding this comment.
@coderabbitai yes。issue作成をお願い。
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Create a GitHub issue to track the skipLibCheck technical debt
gh issue create \
--title "技術的負債: tsconfig.jsonのskipLibCheckを削除する" \
--body "## 概要
openci-runner/github-apps/tsconfig.jsonに\`skipLibCheck: true\`が追加されましたが、これは依存関係の型チェックをスキップするため、依存関係の型エラーを隠蔽する可能性があります。
## 背景
PR #465での大規模な依存関係更新時に、ビルド時間を改善するための一時的な回避策として追加されました。
## 対応内容
すべての型定義が正しいことを確認した後、この設定を削除する必要があります。
## 関連リンク
- PR: https://github.com/open-ci-io/openci/pull/465
- コメント: https://github.com/open-ci-io/openci/pull/465#discussion_r2403787258
cc: @mafreud" \
--assignee mafreud
Length of output: 47
@mafreud issueを作成しました!
📝 Issue #466: 技術的負債: tsconfig.jsonのskipLibCheckを削除する
このissueには以下の情報が含まれています:
skipLibCheck: trueの追加背景と影響- 削除前に確認すべき内容
- このPRとコメントへのバックリンク
担当者として設定されていますので、型定義の確認が完了したら対応をお願いします。
(ﾉ◕ヮ◕)ﾉ*:・゚✧
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
fix: add skipLibCheck to tsconfig.json for improved type checking performance
feat: add 'smee' to workspace settings in openci.code-workspace
Summary by CodeRabbit