Skip to content

local キーワード 27 箇所の扱い方針決定(shebang 変更 or 疑似スコープ化) #46

@ikeisuke

Description

@ikeisuke

背景

Unit 006 コードベース調査レポート(サイクル v0.3.0)の調査候補 #19(観点 7.1)。Unit 008 のユーザーレビューで「起票する」として採否確定。

  • 対象ファイル/箇所: 以下 7 ファイル計 27 箇所
    • lib/aws.sh@bd55d9c: 6 箇所
    • lib/token.sh@bd55d9c: 9 箇所
    • lib/sandbox.sh@bd55d9c: 2 箇所
    • lib/platform/sandbox-darwin.sh@bd55d9c: 2 箇所
    • lib/platform/sandbox-linux-apparmor.sh@bd55d9c: 2 箇所
    • lib/platform/sandbox-linux-systemd.sh@bd55d9c: 3 箇所
    • lib/platform/git-worktree.sh@bd55d9c: 3 箇所
  • 現状の問題: local は POSIX sh で未規定(bash / dash / busybox sh の拡張)。#!/bin/sh shebang と整合しない
  • 推奨対応: (a) #!/bin/bash に shebang を変更して bash 前提を明示する、または (b) local を排し、関数内変数を _FN_varname のような名前規約で疑似スコープ化する。プロジェクトとして POSIX 準拠を維持するなら (b) が望ましい
  • 想定影響範囲: 該当 7 ファイル。tests/posix_compliance.bats の判定基準との整合が必要
  • 確度区分: confirmed

参照

  • 分析対象コミット SHA: bd55d9cb48749fb1df884880b46223a9ff7a1ea8
  • Unit 006 観点別本文: .aidlc/cycles/v0.3.0/plans/code_review_report.md の観点 7.1 セクション
  • Unit 008 採否確定: 同レポート末尾「確定済み起票候補」表 Migrate config to TOML format with Python parser #19

サイクル

v0.3.0 サイクルの Construction Phase(Unit 006 → Unit 008)で採否確定した起票候補。実装修正は別サイクル(次サイクル以降)で対応予定。

Metadata

Metadata

Assignees

No one assigned

    Labels

    backlogバックログアイテムcycle:v0.3.0サイクル v0.3.0priority:high優先度: 高status:backlogバックログにある未着手の状態type:refactorリファクタリング

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions