Skip to content

Disable runtime actions when disconnected - #157

Open
asthasinghcs wants to merge 3 commits into
0-Shimanshu:mainfrom
asthasinghcs:fix-action-availability
Open

Disable runtime actions when disconnected#157
asthasinghcs wants to merge 3 commits into
0-Shimanshu:mainfrom
asthasinghcs:fix-action-availability

Conversation

@asthasinghcs

@asthasinghcs asthasinghcs commented May 30, 2026

Copy link
Copy Markdown
Contributor

Summary

Disable runtime actions when the application is disconnected.

Added action availability handling for the Events and Audit screens and added test coverage for disconnected-state behavior.

Why

Runtime-dependent actions should not appear available when there is no active runtime connection. This change ensures those actions are disabled and provide clear feedback to the user.

Scope

  • bug fix
  • documentation
  • tests
  • TUI / CLI improvement
  • framework integration
  • signal change
  • configuration or route-policy change

Verification

How did you verify the change?

Commands run:

python -m compileall adiuvare
pytest tests/test_tui.py

Notes:

  • I ran the relevant tests locally
  • I updated docs if behavior changed
  • I updated screenshots if the TUI changed materially

Signal-specific notes

Signal type:

  • hard signal (trackA)
  • soft signal (trackB)
  • not applicable

Source, reproducer, or proof:

N/A

Benign case checked:

N/A

Performance note:

N/A

TUI notes

  • before/after screenshots attached
  • not applicable

Events screen (after)

Disconnected state showing runtime actions disabled and unavailable reasons displayed in the status panel.

image

Audit screen (after)

Disconnected state showing runtime actions disabled and unavailable actions visually indicated.

image

Local-only file check

  • I did not accidentally stage local-only files such as adiuvare.yaml or local conftest.py changes

Related issue

Closes #11

@github-actions github-actions Bot added the enhancement New feature or request label May 30, 2026
@asthasinghcs
asthasinghcs force-pushed the fix-action-availability branch from 6732e8b to 671e333 Compare May 30, 2026 18:55
@asthasinghcs

Copy link
Copy Markdown
Contributor Author

Implemented the requested runtime-disconnected action handling and added test coverage for the affected behavior.

All checks are passing from my side now (11/11).

@0-Shimanshu

Copy link
Copy Markdown
Owner

@asthasinghcs follow proper pr template

@0-Shimanshu
0-Shimanshu self-requested a review May 31, 2026 08:47
@0-Shimanshu

Copy link
Copy Markdown
Owner

@asthasinghcs

Copy link
Copy Markdown
Contributor Author

hi @0-Shimanshu i have updated the PR description to follow the template and linked the related issue.

@0-Shimanshu

Copy link
Copy Markdown
Owner

@asthasinghcs you need to attach ss of the change you made in the tui
read the template

@asthasinghcs

Copy link
Copy Markdown
Contributor Author

Hi @0-Shimanshu, I have updated the PR description, attached screenshots of the Events and Audit disconnected-state UI, and verified that all checks are passing on my side.

@asthasinghcs

Copy link
Copy Markdown
Contributor Author

Hi @0-Shimanshu,

Just following up on this PR. I have addressed the requested changes, updated the PR description, attached the required screenshots, and all checks are passing.

Whenever you have time, I'd appreciate a review.

Thanks.

@asthasinghcs

Copy link
Copy Markdown
Contributor Author

Hi @0-Shimanshu , kindly look into this whenever you have time, and if there are any further updates needed.

@asthasinghcs

Copy link
Copy Markdown
Contributor Author

hi @0-Shimanshu kindly look into this, whenever you have time. thank you!

@asthasinghcs

Copy link
Copy Markdown
Contributor Author

hi @0-Shimanshu and @Wahid7852 kindly give this a look, whenever y'all have time :)

@Wahid7852

Copy link
Copy Markdown
Collaborator

Looks good to me overall.

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

tui: make disconnected operator actions more explicit and safer

3 participants