Fix Multi AI Audit consensus findings (12 issues)#4
Conversation
Address all consensus findings (2+ AI agreement) from Multi AI Audit: - [High] Fix run_app.sh: use PYTHONPATH + python3 -m for proper module loading - [High] Fix build_deb.sh launcher: same PYTHONPATH + -m approach - [Medium] build_deb.sh: read version from pyproject.toml, add python3-pyqt6 dep - [Medium] Fix _refresh_after_write: unconditionally refresh on DML - [Medium] Improve destructive query detection: strip string literals before WHERE check - [Medium] Add error path tests (empty query, invalid SQL, no connection, bad path) - [Medium] Fix resources path: avoid as_file() context escape - [Low] Remove unused imports (field, Path, Optional) - [Low] Optimize SQL highlighter: consolidate 96 regex patterns into one - [Low] Remove placeholder domain example.com Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! このプルリクエストは、Multi AI Audit (Claude + Gemini + Codex) によって特定された12件のコンセンサス所見に対処します。主な目的は、アプリケーションの堅牢性を向上させ、ビルドプロセスを合理化し、SQL解析の正確性を高め、UIのリフレッシュ動作を改善することです。これにより、潜在的なバグが修正され、コードベースの全体的な品質と保守性が向上します。 Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Changelog
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: fd01680ac3
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
scripts/build_deb.sh
Outdated
| BUILD_DIR="$DIST_DIR/deb_build" | ||
| PACKAGE="sqliteview" | ||
| VERSION="0.2.1" | ||
| VERSION=$(python3 -c "import tomllib; print(tomllib.load(open('$ROOT_DIR/pyproject.toml','rb'))['project']['version'])") |
There was a problem hiding this comment.
The new VERSION=$(python3 -c "import tomllib; ...") line requires tomllib, which is only available in Python 3.11+, so scripts/build_deb.sh now fails immediately on Python 3.10 build hosts with ModuleNotFoundError. This is a regression against the repository/package requirement of python >= 3.10, and it blocks generating the Debian package in a supported environment (e.g., Ubuntu 22.04 default Python).
Useful? React with 👍 / 👎.
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
このプルリクエストは、AI監査によって検出された12件の問題を修正し、動的なバージョン取得、依存関係の追加、SQLインジェクションの脆弱性修正、UIの更新ロジック改善、テストの追加を通じて、コードの堅牢性を大幅に向上させています。しかし、database.pyにおける破壊的なクエリチェックをバイパスする修正は不完全であり、単一引用符の文字列のみを処理し、二重引用符の識別子、バッククォート、角括弧など、SQLiteステートメントにキーワードを埋め込む他の方法を見落としているため、依然として重要なセキュリティ脆弱性が残っています。さらに、ビルドプロセスとリソースの読み込みに関して2つの潜在的な問題が特定されました。これらの点を修正することで、アプリケーションの安定性がさらに向上するでしょう。
scripts/build_deb.sh
Outdated
| BUILD_DIR="$DIST_DIR/deb_build" | ||
| PACKAGE="sqliteview" | ||
| VERSION="0.2.1" | ||
| VERSION=$(python3 -c "import tomllib; print(tomllib.load(open('$ROOT_DIR/pyproject.toml','rb'))['project']['version'])") |
There was a problem hiding this comment.
この変更では tomllib を使って pyproject.toml からバージョンを取得していますが、tomllib は Python 3.11 で標準ライブラリに追加されたモジュールです。pyproject.toml の requires-python は >=3.10 となっており、Python 3.10 の環境でこのビルドスクリプトを実行すると ModuleNotFoundError: No module named 'tomllib' エラーで失敗してしまいます。
Python 3.10 との互換性を保つために、tomli パッケージ（tomllib の前身）をフォールバックとして使用することをお勧めします。以下のように一行で修正できます。
| VERSION=$(python3 -c "import tomllib; print(tomllib.load(open('$ROOT_DIR/pyproject.toml','rb'))['project']['version'])") | |
| VERSION=$(python3 -c "try: import tomllib except ImportError: import tomli as tomllib; print(tomllib.load(open('$ROOT_DIR/pyproject.toml','rb'))['project']['version'])") |
Past PRs で繰り返し発生した修正パターンをルール化: - 未使用 import の即時削除 - セキュリティチェックの初回網羅実装 - python3 -m によるランチャー起動 - pyproject.toml を SSOT としたバージョン管理 - 変更の波及先確認チェックリスト - uv によるPython環境統一 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- build_deb.sh: tomllib (Python 3.11+) を grep/sed に置換して Python 3.10 互換性を復元 - database.py: _strip_sql_noise でダブルクォート識別子も除去対象に追加 - test_database.py: ダブルクォート識別子内の WHERE キーワードに対するテスト追加 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- セキュリティチェックルールにダブルクォート識別子を追加 - バージョン取得ルールにPython最低バージョン互換性の注意を追加 - PLAN.mdの記述を実態に合わせて修正 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 participant
Summary
Multi AI Audit (Claude + Gemini + Codex) で検出された Consensus findings 12件を修正。
High
PYTHONPATH設定 +python3 -m sqliteviewerに変更（相対インポート失敗を修正）-m起動に修正Medium
pyproject.tomlから動的取得、python3-pyqt6依存追加_refresh_after_writeでサブストリングマッチ廃止→DML後は無条件リフレッシュ_strip_sql_noiseに拡張し文字列リテラル内 WHERE によるバイパスを防止as_file()コンテキスト外パス参照問題を修正Low
fieldimport 削除Path,Optionalimport 削除example.com削除Test plan
uv run python -m pytest tests/ -v— 全15テスト pass🤖 Generated with Claude Code