Skip to content

firmware_install docstring still describes version-only after_confirmed semantics #119

Description

@millsmillsymills

Follow-up from review of #114 (merged as #117).

#117 changed the after_confirmed gate in _reconnect_and_classify from version-only to (version, flavor) identity change, and updated that helper's docstring — but the public tool docstring was not updated. src/flipperzero_mcp/tools/firmware.py (flipperzero_firmware_install, Returns block) still says:

after_confirmed (True only if a post-reboot version change was observed; ...)

An agent reading the tool description will wrongly conclude a same-version fork swap reports after_confirmed=False. Update the Returns block to say the confirm fires on a post-reboot (version, flavor) identity change against a known baseline.

One-line docstring fix; no behavior change, no new tests needed.

🤖 Generated with Claude Code

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions