Repository navigation
Conversation
Signed-off-by: Deepak Jain <deepujain@gmail.com>
|
Full qualification is currently blocked by existing main-branch CI failures. Linked prerequisite: #508 fixes the enterprise-disallowed notification action in the test workflow and stale HTTPResponseError assertions after merged #484. The SSH repair remains scoped to its two files. GitHub has held the fork workflows for maintainer approval. A maintainer with Actions write authority needs to approve the runs on #508 and this PR. My account has read-only upstream access, so it cannot approve those runs or the code-owner review. No unchanged workflow reruns or new repair pushes have been triggered. Published SSH candidate: |
When the Brev SSH Include is inside an unrelated
HostorMatchstanza,brev refreshcurrently finds its text and leaves the alias unresolved. Only an Include in global scope now counts as installed, so the existing insertion path prepends a global Include. The existing configuration is preserved, and subsequent refreshes do not rewrite it again. Unix and Windows path matching remain supported.Related to NVIDIA/NemoClaw#13090 and its QA reproduction. This does not establish who originally misplaced the workstation Include.
Validation dependency: #508 repairs the base's disallowed CI notification action and stale assertions after merged #484. Full qualification requires that dependency and maintainer approval of fork workflows.
Validation:
ssh -F ... -Grouting recovery, unrelated settings, Unix/WSL idempotence, and certificate-template coverage. New regressions fail against unchanged basea0190d24655f57f327778f7708d8a6f634afdb1a.The retained QA workstation still needs the corrected CLI tested through both original Setup over SSH and Coding Agent paths, refresh/reconnect, onboarding and a real agent response. This PR does not claim production acceptance or publish a CLI release.