HYP-147: Ignore generated lint artifacts#416
Conversation
proboscis
left a comment
proboscis
left a comment
There was a problem hiding this comment.
結論: code-review は現時点では通せません。PR の向き先が main であることは確認できましたが、現在の main を取り込んだ状態では HYP-147 の repo hygiene 対象がまだ残ります。
確認したこと:
- PR #416 は
main向けで、lint ログ、doeff-flow の DB / trace、None/Untitledの削除とmake lint-doeffの--no-log化は意図に沿っています。 - PR branch 単体では
make check-repo-hygiene、変更した example の ruff / format check / py_compile は通っています。 - ただし current
mainとの仮想 merge 後も、.agent-home/.claude*.json、.agent-home/.claude/config.json、.agent-home/.claude/settings.json、.playwright-mcp/page-*.yml、.agents/skills/install-vscode-plugin/SKILL.md、rootserver.jsが追跡対象に残ります。 - 追加された
scripts/check-repo-hygiene.shは DB / SQLite / JSONL /None/Untitledだけを見るため、上記の agent/session 系 artifact の再追跡を検出できません。
お願い:
- PR を current
mainに更新したうえで、上記 artifact を削除・ignore・明示 fixture 化のいずれかに分類してください。 - 分類後、
check-repo-hygieneもその分類を検査できるように広げてください。fixture として残すものがある場合は、参照元と許可リストを明示してください。
|
確認結果: 対象外です。 current head そのため、 |
|
結論: PR #416 の current head
このため、この確認では修正要求はありません。 |
|
設計方針確認: 対象外です。 現行 PR head: 確認結果:
この確認では修正要求はありません。 |
|
UI 証跡確認: 対象外です。 PR #416 の current head そのため UI 証跡画像は不要で、 |
|
PR説明図チェック: 通過です。 確認結果:
このチェックとして追加修正は不要です。 |
|
doeff-hy-style 確認結果: 対象外です。 current head このため、 |
|
UI 証跡確認: 対象外です。 current head UI、画面レイアウト、ナビゲーション、スクリーンショット表示状態、UI 上の文言変更はありません。PR 説明図は別チェック対象であり、今回の UI 証跡判定には使っていません。 そのため UI 証跡画像は不要で、 |
|
設定と効果境界の確認結果: 対象外です。 current head 追加・変更された実行コードでは、設定注入、環境変数読み取り、 |
|
設計方針確認（core-philosophy）: 対象外です。 現在の PR head: 確認結果:
記録結果は |
|
PR説明図チェック: 通過です。 確認結果:
このチェックとして追加修正は不要です。 |
|
結論: code-review はまだ通せません。PR は 確認したこと:
修正してほしいこと:
この修正が入れば、HYP-147 の code-review としては通せる見込みです。UI、PR説明図、Hy style、設定/effect 境界の詳細確認は別チェックに委ねています。 |
config-effects 確認結果結論: 対象外です。PR #416 は生成物の追跡解除、 確認したこと:
このため |
|
結論: code-review は通過です。 確認したこと:
補足:
この確認では追加修正要求はありません。 |
|
doeff-hy-style 確認結果: 対象外です。 current head |
|
設計方針確認（core-philosophy）: 対象外です。 現在の PR head: 確認結果:
このチェックは |
|
UI 証跡確認: 対象外です。 PR #416 の現在 head
|
|
PR説明図確認: 通過です。
|
|
config-effects 確認結果: 対象外です。 current head: 今回の差分は そのため、この review check は |
|
doeff Hy style 確認結果: 対象外です。 現在の PR head |
|
設計方針確認（core-philosophy）は対象外です。 PR head: 確認結果:
この確認は |
proboscis
left a comment
proboscis
left a comment
There was a problem hiding this comment.
結論: code-review は通過です。
確認したこと:
- PR #416 は
main向けで、current head の差分を確認しました。 make lint-doeffはdoeff-linter --no-logを渡す形に変更されており、標準 lint 経路で.doeff-lint.jsonlを生成しない方針になっています。- 追跡済みだった lint log、doeff-flow の DB / trace、
None/Untitled、agent 実行状態、Playwright capture、ローカル Claude 設定 / skill、rootserver.jsは追跡対象から外れています。 - 前回の差し戻し対象だった
*.sqlite-wal/*.sqlite-shmは.gitignoreとscripts/check-repo-hygiene.shの両方に入り、一時 index で追跡対象にした場合に検出されることを確認しました。 make check-repo-hygiene、bash -n scripts/check-repo-hygiene.sh、git diff --check、変更した doeff-flow example の ruff / py_compile は通過しました。
補足: この環境では doeff-linter が未インストールのため、make lint-doeff は既存の警告のみです。UI 証跡、PR説明図、Hy style、設定 / effect 境界の詳細確認は別チェックに委ねています。
この確認では追加修正要求はありません。
|
結論: doeff Hy style check は対象外です。 確認内容:
記録: |
|
設計方針確認（core-philosophy）: 対象外です。 確認した現在 head: この PR の差分は repo hygiene のための生成物削除と再混入検査が中心です。具体的には 設計方針確認の対象である |
|
config-effects 確認結果: 対象外です。 Current head 設定注入、実行時設定の読み込み、環境変数参照、interpreter setup、 このため、この確認は |
proboscis
left a comment
proboscis
left a comment
There was a problem hiding this comment.
結論: code-review は通過です。
確認したこと:
- PR #416 は
main向けで、現在の head の差分を確認しました。 make lint-doeffはdoeff-linter --no-logを渡す形で、標準 lint 経路が.doeff-lint.jsonlを生成しない構成になっています。- 追跡済みだった lint log、doeff-flow の DB / trace、
None/Untitled、agent 実行状態、Playwright capture、ローカル Claude 設定 / skill、rootserver.jsは追跡対象から外れています。 .gitignoreとscripts/check-repo-hygiene.shは SQLite DB と WAL / SHM 側ファイル、JSONL、ローカル agent / browser automation state を検査対象に含めています。- 一時 index で
*.sqlite-wal/*.sqlite-shmを追跡対象にした場合も、repo hygiene 検査が期待通り検出することを確認しました。 make check-repo-hygiene、bash -n scripts/check-repo-hygiene.sh、git diff --check、変更した doeff-flow example の ruff / py_compile は通過しました。
補足: この環境では doeff-linter が未インストールのため、make lint-doeff は既存の警告のみです。make check は既存の repo 全体の ruff format drift で止まりましたが、今回の変更ファイル起因ではないと判断しています。UI 証跡、PR説明図、Hy style、設定 / effect 境界の詳細確認は別チェックに委ねます。
この確認では追加修正要求はありません。
|
PR説明図確認: 通過です。 確認結果:
この確認では追加修正要求はありません。 |
1 participant
概要
HYP-147 の実装更新です。通常の lint や doeff-flow サンプル実行で生成されるログ、SQLite DB、trace に加えて、current
mainに残っていた agent 実行状態、Playwright MCP 画面キャプチャ、ローカル Claude 設定 / skill、rootserver.jsを git の追跡対象から外し、再混入を検査する経路を更新しました。今回の差し戻し対応として、SQLite の WAL / SHM 側ファイルのうち漏れていた
*.sqlite-walと*.sqlite-shmも.gitignoreとmake check-repo-hygieneの検査対象に追加しました。変更内容
make lint-doeffがdoeff-linter --no-logを使うように変更。.gitignoreに linter ログ、doeff-flow trace、SQLite 系ファイル、WAL / SHM 側ファイル、None/Untitled、.agent-home/、.playwright-mcp/、ローカル Claude 設定 / skill、rootserver.jsを追加。.doeff-lint.jsonl、None、Untitled、doeff-flow の trace / DB、.agent-home/、.playwright-mcp/、.claude/settings.local.json、.claude/skills/install-vscode-plugin/、.agents/skills/install-vscode-plugin/、rootserver.jsを削除。05_durable_execution.pyの SQLite DB を.doeff-flow/durable_workflow.dbに移し、README のサンプル一覧も更新。make check-repo-hygieneの検査対象に DB / SQLite / JSONL / 一時ファイルだけでなく、WAL / SHM 側ファイル、agent 実行状態、画面キャプチャ、ローカル設定、rootserver.jsを追加。確認
make check-repo-hygiene: 通過。bash -n scripts/check-repo-hygiene.sh: 通過。git ls-filesによる生成物検査: 出力なし。*.sqlite-walを追跡対象にした検査: 修正前は見逃し、修正後は検出。make lint-doeff: この環境ではdoeff-linter未インストールの既存警告のみ。target は--no-logを渡す形に更新済みで、linter ログは作成されません。git diff --check: 通過。