scripts: shell: Fix bluetooth shell

[bao-eng]

Restored the working order of the Bluetooth LE Console application.

[CLAassistant]

All committers have signed the CLA.

[NordicBuilder]

Thank you for your contribution!
It seems you are not a member of the nrfconnect GitHub organization. External contributions are handled as follows:
Large contributions, affecting multiple subsystems for example, may be rejected if they are complex, may introduce regressions due to lack of test coverage, or if they are not consistent with the architecture of nRF Connect SDK.
PRs will be run in our continuous integration (CI) test system.
If CI passes, PRs will be tagged for review and merged on successful completion of review. You may be asked to make some modifications to your contribution during review.
If CI fails, PRs may be rejected or may be tagged for review and rework.
PRs that become outdated due to other changes in the repository may be rejected or rework requested.
External contributions will be prioritized for review based on the relevance to current development efforts in nRF Connect SDK. Bug fix PRs will be prioritized.
You may raise issues or ask for help from our Technical Support team by visiting https://devzone.nordicsemi.com/.

Note: This comment is automatically posted and updated by the Contribs GitHub Action.

[nordic-krch]

@bao-eng thanks for that patch. Can you add fix description to commit message?

[bao-eng]

@nordic-krch description added, please review.

[bao-eng]

@nordic-krch @nordicjm Fixed a few more things to pass compliance checks.
Please approve.

[bao-eng]

@nordic-krch @nordicjm
It looks like the compliance check fails because the runner is missing the PyGObject package:
https://github.com/nrfconnect/sdk-nrf/actions/runs/13772628780?pr=20805

E0611:No name 'Gtk' in module 'gi.repository' (no-name-in-module)
File:scripts/shell/ble_console/BluetoothConsole.py
Line:12
Column:0
 E0611:No name 'Gtk' in module 'gi.repository' (no-name-in-module)
File:scripts/shell/ble_console/TerminalNotebook.py
Line:10
Column:0
 E0611:No name 'Gdk' in module 'gi.repository' (no-name-in-module)
File:scripts/shell/ble_console/TerminalNotebook.py
Line:10
Column:0
 E0611:No name 'Vte' in module 'gi.repository' (no-name-in-module)
File:scripts/shell/ble_console/TerminalNotebook.py
Line:13
Column:0
 W0612:Unused variable 'dir_name' (unused-variable)
File:scripts/shell/ble_console/TerminalNotebook.py
Line:27
Column:12

I added requirements.txt, but I am unsure if it will resolve this issue.
Any advice?

[bao-eng]

@nordic-krch @nordicjm
Well, now, compliance fails due to this:

Error: New files added that are not covered in CODEOWNERS:

scripts/shell/ble_console/requirements.txt

Please add one or more entries in the CODEOWNERS file to cover those files

Please help.

[nordic-krch]

You can add me:

--- a/CODEOWNERS
+++ b/CODEOWNERS
@@ -706,6 +706,7 @@
 /scripts/nrf_provision/fast_pair/*.rst    @nrfconnect/ncs-si-bluebagel-doc
 /scripts/partition_manager/*.rst          @nrfconnect/ncs-aurora-doc
 /scripts/shell/ble_console/**/*.rst       @nrfconnect/ncs-doc-leads
+/scripts/shell/ble_console/*              @nordic-krch
 /scripts/west_commands/sbom/*.rst         @nrfconnect/ncs-si-muffin-doc

[jangalda-nsc]

@bao-eng could you please cherrypick following commit?
371e346
It should help with compliance issue

[bao-eng]

@jangalda-nsc thank you!
@nordic-krch added you as an owner, please approve

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[bao-eng]

@nordic-krch apparently, warnings are also critical; fixed the last one

[nordicjm]

@bao-eng needs rebase