Skip to content

add CLI interface for USBCopySession with resume() support - #6

Merged
satoshi-kondo merged 4 commits into
mainfrom
feature/usb-copy-cli
Dec 3, 2025
Merged

add CLI interface for USBCopySession with resume() support#6
satoshi-kondo merged 4 commits into
mainfrom
feature/usb-copy-cli

Conversation

@satoshi-kondo

Copy link
Copy Markdown
Contributor

Issue リンク / Link to Issue

https://github.com/airoa-org/hsr_devops/issues/114

やったこと / What I did

  • USBCopySession に対して CLI のインターフェースを用意。

やってないこと / What I did not

  • 上記以外

チェック観点 / Check Points

  • コード詳細を確認いただくというより docs/cli-usage.md を参考に実際にコピー&アップロードする環境でコマンドが実行できそうか?を確認していただければと思います。

補足 / Supplement

  • 今回の実装で問題なさそうであれば、WasabiUpload についても同様の実装を実施いてきます。

@yuya-haruna yuya-haruna left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

コメントしています!

Comment thread pyproject.toml Outdated
]

[project.scripts]
airoa-usb-copy = "airoa_lineage.cli.usb_copy:main"

@yuya-haruna yuya-haruna Dec 1, 2025

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

コマンド名の意味が広すぎるかなと思いました。
長くなってしまいますが、 airoa-lineage-usb-copy などはいかがでしょうか。

airoa-usb-copyだと、usbにcopyするコマンドのように見えてしまい)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

以下のコミットで対応しました。
f4b2389

help="Show what would be done without executing",
)

# COMPLETE command

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

STARTと同様に、必要なFacetsは全てコマンドライン引数で渡せるようにしていただけますでしょうか。

このCLIを叩く、autouploaderはREADME.mdを読む限りバイナリを配布しているような感じらしく、コミットのハッシュ値などをアップデートのたびにconfigに反映してもらうのは運用的に難しいかなと思いました。

そのため、バイナリに必要な値は埋め込んでおいて、それをcliに直接渡して通知できる仕様の方が良い気がしました。
(configの機能自体は持っておいても良いとは思います)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

以下のコミットで対応しました。
a5e216d

@satoshi-kondo

Copy link
Copy Markdown
Contributor Author

@yuya-haruna
指摘があった 2点について対応しました。
また、CLI についても以下のコミットでサンプルコードの追加を行っています。
28cd653

@yuya-haruna yuya-haruna left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@satoshi-kondo
satoshi-kondo merged commit b57d8aa into main Dec 3, 2025
1 check passed
@satoshi-kondo
satoshi-kondo deleted the feature/usb-copy-cli branch December 25, 2025 07:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants