Skip to content

Stabilize termscp for 1.0.0#407

Merged
veeso merged 15 commits into
1.0.0from
refactor/stabilize-termscp
Mar 21, 2026
Merged

Stabilize termscp for 1.0.0#407
veeso merged 15 commits into
1.0.0from
refactor/stabilize-termscp

Conversation

@veeso
Copy link
Copy Markdown
Owner

@veeso veeso commented Mar 21, 2026

Summary

  • stabilize core error handling by removing production panic/unwrap paths and normalizing localhost path handling
  • split oversized auth/parser modules into focused submodules and add core/module API documentation
  • add targeted regression coverage for parser, config, explorer, localhost, bookmarks, auto-update, and ssh key storage paths
  • enable a safe clippy pedantic subset and fix the resulting low-risk diagnostics while keeping cargo clippy --no-default-features -- -Dwarnings green

Verification

  • cargo +nightly fmt --all
  • cargo build --no-default-features
  • cargo clippy --no-default-features -- -Dwarnings
  • targeted test runs for parser, config::serialization, explorer::formatter, host::localhost, and system::sshkey_storage modules

Notes

  • the known flaky network probe test system::auto_update::test::test_should_check_whether_github_api_is_reachable still fails in restricted environments and was not used as a merge gate
  • the original pedantic plan item was reduced to the lints that were safe on the current tree; needless_pass_by_value and semicolon_if_nothing_returned were intentionally not enabled because they exploded into broad UI refactors rather than a safe stabilization pass

@cocogitto-bot
Copy link
Copy Markdown

cocogitto-bot Bot commented Mar 21, 2026

✔️ 43f2f8f...65f24c1 - Conventional commits check succeeded.

@veeso veeso added this to the 1.0.0 milestone Mar 21, 2026
@veeso veeso merged commit 1acb8b2 into 1.0.0 Mar 21, 2026
5 checks passed
@veeso veeso deleted the refactor/stabilize-termscp branch March 21, 2026 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant