Repository navigation
docs: スプリント m10a の確定仕様化(開発サービス名の読み方・決定 6 の書き直し) - #443
Merged
Merged
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01436uwddpXYVHsfZa3f8fML
base-image-chain-build.md の決定 6 は「get_dev_service_name() は空の値をそのまま返す」を 理由にしていた。#425 で Python も未設定か空なら dev と読むようになったため、--service で渡す 形を保つ理由を「読み方が将来分かれても揃う」へ書き直す。あわせて snapshot-series.md の クラス図の rotate の戻り値を RotateResult に、隔離の検査の出力例の行番号を今の行に直す。 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01436uwddpXYVHsfZa3f8fML
Merged
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
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.
Pull Request
Summary
スプリント m10a(#442 でマージ済み)の確定仕様化。#425 の要求・設計を確定仕様
docs/specifications/dev-service-name.mdへ移し、issues/の元を消す。あわせて、既存の確定仕様のうち m10a の実装と食い違う記述を実装に合わせる(コードは変えない)。関連 Issue
変更点
docs/specifications/dev-service-name.md(開発サービス名の読み方): 読み方の表(未設定・空 →dev、空白だけ・それ以外 → 値のまま)、常に成り立つ条件 I1〜I4、設計の判断の結論と理由、テスト観点。
issues/issue-425-requirements.md・issues/issue-425-design.mdは削除base-image-chain-build.mdの決定 6: 理由にしていた「get_dev_service_name()は空の値をそのまま返す」は DEV_SERVICE_NAME を空で書くと、bin/devbase は dev を使い Python は空の名前で開発サービスを探す #425 で事実でなくなった。2 つの読み方は今は揃っているが、将来分かれても建てるサービスと場所を決めるサービスが分かれないよう
--serviceで渡す形を保つ、と書き直した。概要の「既定
dev」も「未設定か空ならdev」に揃え、関連リンクに新しい仕様を足したsnapshot-series.md: クラス図のrotate(keep, max_total)の戻り値をintからRotateResultに直した(snapshot rotate が世代を消したとき、場所の不正で外した数を要約に出さない #333 の実装、本文の手順 6 はすでにRotateResult)test-environment-isolation.md: 隔離の検査の出力例のDEV_SERVICE_NAME: volume/compose.py:37を今の読み取りの行 41 に直したsecret-backend.md・architecture.md・test-environment-isolation.mdのenv_delegate.py)は各実装 PR で済んでおり、消えた関数名(
_group_attrs・_run_in_project・_dispatch・is_groupedなど)がdocs/に残っていないことを確かめたdev」に直してある)利用者向けの変化
移行の手順
Test plan
env -u DEVBASE_ROOT uv run --locked pytest tests/ -q→ exit=0(4113 passed, 20 skipped)uvx ruff check --select=E9,F63,F7,F82 lib→ exit=0python3 .github/scripts/proper_term_check.py→ exit=0。ただし「語の一覧が無いため飛ばした」と出ており、検査はしていないglossary.py check --root .→ exit=0(当たりなし)doc-lint.py --base origin/main→.ndf/project.jsonのndf_policies.doc_lintが true でないため掛けていないlib/devbase/volume/compose.pyのget_dev_service_name()(os.environ.get('DEV_SERVICE_NAME') or 'dev')、tests/volume/test_compose_dev_service_name.py(4 値の単体とgenerate_scaled_composeの経路)、tests/test_env_isolation.pyのtest_collector_finds_known_reads・test_test_side_setenv_wins、lib/devbase/snapshot/manager.pyのRotateResult補足
開発者向けの文書だけの変更のため CHANGELOG は書かない。
🤖 Generated with Claude Code
https://claude.ai/code/session_01436uwddpXYVHsfZa3f8fML
モード: standard / 通した工程: 確定仕様化 → Pull Request