feat(bcl): return BCT_ACK from BCL_tool_register_* functions

[Toshihiro Suzuki (suzuki-toshihir0)]

Summary

• BCL_tool_register_cmd などの BCL 公開 API 7関数の戻り値を void → BCT_ACK に変更した。
• BCT_register_cmd の登録失敗（BCT_INVALID_CMD_NO）をユーザーが BCL_tool_register_hoge のような関数で検知できるようになる。

Background

• BCT_register_cmd はコマンド登録数が BCT_MAX_CMD_NUM を超えると BCT_INVALID_CMD_NO を返してサイレントに失敗する。従来の BCL 公開 API はすべて void 型だったため、この失敗に気づく手段がなかった。
• なので、Task Listなどを書くときに BCT_MAX_CMD_NUM を超える数のコマンドを詰めてしまっても気づきにくい。

Design Notes

• 後方互換性: C では戻り値の無視は合法なため、既存の BCL_tool_register_cmd(ti, id); という呼び出しはそのまま動作する
• EL recordはユーザー任せにする: エラー検知後の EL 記録はユーザーが自分のユーザー定義グループを使って行う

🤖 Generated with Claude Code

[Copilot]

Pull request overview

This PR updates the Block Command Loader (BCL) public API to propagate error status from underlying BCT registration functions. Previously, all 7 BCL public API functions (BCL_tool_register_cmd, BCL_tool_register_cmd_to_other_obc, BCL_tool_register_rotate, BCL_tool_register_combine, BCL_tool_register_limit_combine, BCL_tool_register_deploy, BCL_tool_register_app) had void return types, making it impossible for callers to detect when BCT_register_cmd fails (e.g., when the command table is full and returns BCT_INVALID_CMD_NO). Now these functions return BCT_ACK to enable error detection while maintaining backward compatibility.

Changes:

• Changed return type from void to BCT_ACK for 7 BCL public API functions
• Updated 3 internal static helper functions to return BCT_ACK
• Modified all function implementations to capture and return the BCT_ACK status from BCT_register_cmd

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
tlm_cmd/block_command_loader.h	Updated function declarations and documentation to return BCT_ACK instead of void for 7 public API functions
tlm_cmd/block_command_loader.c	Modified implementations to capture BCT_ACK from underlying calls, updated internal static function signatures

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Ryo Suzumoto (meltingrabbit)]

よさそう