Skip to content

Improve session capability detection - #118

Merged
bifrost0x merged 1 commit into
mainfrom
dev/discussion-116-capability-aware-sessions
Aug 17, 2026
Merged

Improve session capability detection#118
bifrost0x merged 1 commit into
mainfrom
dev/discussion-116-capability-aware-sessions

Conversation

@bifrost0x

Copy link
Copy Markdown
Owner

What changed

  • collect session metrics with guarded POSIX and BusyBox-compatible probes
  • render only metrics and diagnostic sections that returned usable data
  • keep base telemetry active when optional expanded diagnostics are unsupported
  • probe SFTP capability on a fresh, bounded channel before auto-opening the embedded SFTP pane
  • retry transient SFTP probe failures without treating them as permanent incompatibility
  • preserve early terminal output and switch or appliance prompts during attachment and restore

Why

Servers, switches, and appliances expose different command sets and SSH subsystems. Empty metric panels and an automatically opened but unusable SFTP pane created unnecessary errors for unsupported targets. The session workspace now adapts to the capabilities actually available while keeping probes bounded and read-only.

The standalone SFTP Manager is unchanged, and no SCP fallback is introduced.

Validation

  • pytest tests -q: 1603 passed, 33 skipped
  • npm run test:js: 216 passed
  • npm run test:e2e: 6 passed
  • npm run lint:js
  • npm run vendor:check
  • git diff --check

Implements the compatibility behavior discussed in Discussion #116.

@bifrost0x
bifrost0x marked this pull request as ready for review August 17, 2026 21:32
@bifrost0x bifrost0x self-assigned this Aug 17, 2026
@bifrost0x bifrost0x added bug Something isn't working enhancement New feature or request labels Aug 17, 2026
@bifrost0x
bifrost0x merged commit a12d46b into main Aug 17, 2026
15 of 16 checks passed
@bifrost0x
bifrost0x deleted the dev/discussion-116-capability-aware-sessions branch August 17, 2026 21:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant