Repository navigation
Conversation
|
Rebased onto origin/main and resolved the .github/facts.json conflict by keeping the incoming (PR) generated hashes, since this branch updates docs/commands that change those artifacts. Build verified clean after rebase; force-pushed to dw9-cli-4. |
|
Fixed stale facts.json hashes: ran |
|
Rebased onto origin/main; whole-module tests pass locally. The ratchet failures cited in the stale CI output (package-size for internal/database, godotenv-reads, docker funnel, panic baseline) do not relate to this PR's changes (docs/generated JSON/small cmd/commands fixes). No code fix was required — only the rebase. |
Rollback now resolves inventory credentials and server names within its target environment. Removal flag help now explains the v1.5 data-preservation default and confirmation requirement alongside v1.4 behavior.
inv.Environments[target].Servers; preserve the recorded tar's remote path and fallback credentials. Regression covers two environments sharing a host with different SSH keys/server names, plus a missing target. The naming prerequisite was already settled in feat(deploy): v1.5 remote apply ships the artifact set, previews recreates, backs up first (P7-DEPL-24) #629.d3444e27(test: macOS/Linux conformance (P7-HYG-49) #765) changedTestE487HintEnrollThenDeploytopintest.Home(t), which handles unsafe temp ancestors.Validation:
go test -mod=vendor ./cmd/commands -run '^TestRollbackInventoryUsesOnlyTargetEnvironment$' -count=20.go test -mod=vendor ./cmd/commands -run '^(TestInstallFlagParity|TestRemoveDeleteDataHelp|TestPluginRemoveDataPolicyCmd)$' -count=1passed afterward.make cmd-inventory, MCPTestToolListGolden -update,make wiki-commands, andgo run -mod=vendor ./tools/sdkgen -lang ts.make build BINARY="$TMPDIR/nself",go vet -mod=vendor ./cmd/commands,golangci-lint run ./cmd/commands/...(zero issues).go test -mod=vendor -timeout 10m ./internal/mcpgen ./internal/repoqa.make fmt-check,bin/wikigen -check, andgit diff --check.go test -mod=vendor -timeout 10m ./cmd/commandsin 398.960 seconds, with no tests skipped.An initial full command-package run with the system Go 1.27.2 timed out at the ten-minute limit while
TestSurfaceIdentitywas active. Its isolated clean-origin/main comparison at68dc16a0passed in 26 seconds, so that timeout is not claimed as a reproduced baseline failure. Final checks use Go 1.26.10 and task-local caches; this also avoids missing shared-cache files and the installed linter's inability to read Go 1.27.2 export data.No HQ files, production systems, versions, or releases were changed. No hooks were excluded.