Skip to content

fix(smoke): update help screenshot baseline for netclaw memory command - #1603

Merged
Aaronontheweb merged 1 commit into
netclaw-dev:feature/memory-embeddingsfrom
Aaronontheweb:fix/help-screenshot-baseline-memory-cmd
Jul 8, 2026
Merged

fix(smoke): update help screenshot baseline for netclaw memory command#1603
Aaronontheweb merged 1 commit into
netclaw-dev:feature/memory-embeddingsfrom
Aaronontheweb:fix/help-screenshot-baseline-memory-cmd

Conversation

@Aaronontheweb

Copy link
Copy Markdown
Collaborator

Summary

  • Screenshot Regression (Linux) has been failing on every push to feature/memory-embeddings since feat(memory): embedding foundation — ONNX runtime, provisioning, embed-on-write, backfill (memory-core-redesign slice 2) #1577 landed the netclaw memory CLI command — the help screenshot baseline was never updated to include it, so every frame after it shifts down one row and fails the byte-for-byte comparison.
  • Verified this is pre-existing and unrelated to the dev-sync in Merge dev into feature/memory-embeddings #1601: gh run list shows the Screenshot Regression (Linux) job failing on the last 3 pushes to feature/memory-embeddings (including before the dev merge).
  • Ran ./scripts/smoke/run-smoke.sh screenshots locally, reviewed the diff: the only change is the new memory command row in netclaw --help. All 8 other frames (wizard-provider-picker, wizard-security-posture, provider-manager-empty, mcp-permissions-server-list, mcp-permissions-tool-grid, config-search-selection, config-search-brave-entry, config-search-saved) passed with AE=0.
  • Per the documented baseline workflow (tests/smoke/tapes/README.md § Baseline workflow), committing the reviewed help.actual.png as the new approved baseline.

Test plan

  • ./scripts/smoke/run-smoke.sh screenshots run locally against this branch — confirm help now passes and no other frame regresses
  • CI Screenshot Regression (Linux) passes on this PR

The memory-core-redesign slice 2 (netclaw-dev#1577) added the `netclaw memory`
CLI command but never updated the approved `help` screenshot baseline,
so Screenshot Regression (Linux) has been red on every push to
feature/memory-embeddings since (verified via gh run list against
feature/memory-embeddings: 3/3 recent pushes failed this check).

Reviewed the captured diff locally: the only change is the new
  memory              Manage cross-session memory (embeddings backfill, offline)
line shifting everything below it down one row. No other screenshot
frame changed.
@Aaronontheweb
Aaronontheweb merged commit 12e4476 into netclaw-dev:feature/memory-embeddings Jul 8, 2026
14 checks passed
@Aaronontheweb
Aaronontheweb deleted the fix/help-screenshot-baseline-memory-cmd branch July 8, 2026 17:48
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