Skip to content

Conversation

portasynthinca3
Copy link
Member

@portasynthinca3 portasynthinca3 commented Apr 3, 2025

What's new

  • cli_shell and friends have been made more generic and extracted into toolbox, allowing commands to define their own shells
  • Command to test subshells subshell_demo

Verification

  • Run unit tests
  • Verify that the shell still functions as intended (incl. with companion apps)
  • Run subshell_demo, verify that it works as intended

Checklist (For Reviewer)

  • PR has description of feature/bug or link to Confluence/Jira task
  • Description contains actions to verify feature/bugfix
  • I've built this code, uploaded it to the device and verified feature/bugfix

Copy link

github-actions bot commented Apr 3, 2025

PVS-Studio report for commit bef24875:

@portasynthinca3 portasynthinca3 added Core+Services HAL, furi & core system services App Loader FAP loader + API-related labels Apr 3, 2025
Copy link

github-actions bot commented Apr 3, 2025

Compiled f7 firmware for commit d9744e14:

@portasynthinca3 portasynthinca3 marked this pull request as ready for review April 3, 2025 19:28
@hedger hedger merged commit 7192c9e into dev Apr 4, 2025
11 checks passed
@hedger hedger deleted the porta/4147-cli_shell-toolbox-2 branch April 4, 2025 22:58
hedger added a commit to RebornedBrain/flipperzero-firmware that referenced this pull request Apr 5, 2025
* cli_shell: separate into toolbox

* fix: cmd flags

* fix formatting

* cli: increase default stack depth

* cli_shell: fix loader lock logic

* cli: fix command flags

* fix f18

* speaker_debug: fix

* cli_registry: fix docs

* ufbt: rename cli target back

* cli: rename app and record

* cli: fix and simplify help command

* cli_master_shell: fix ext commands

* fix formatting

* cli: rename master to main

* fix formatting

---------

Co-authored-by: hedger <hedger@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

App Loader FAP loader + API-related Core+Services HAL, furi & core system services

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants