Skip to content

test(sap): add reversible standard text lifecycle - #473

Open
HappyDevs1 wants to merge 3 commits into
mainfrom
codex/complex-sap-standard-text-write
Open

test(sap): add reversible standard text lifecycle#473
HappyDevs1 wants to merge 3 commits into
mainfrom
codex/complex-sap-standard-text-write

Conversation

@HappyDevs1

@HappyDevs1 HappyDevs1 commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator

Why

The existing complex SAP example is deliberately read-only. This adds a similarly long scenario that exercises real writes while deleting the sandbox record before the flow finishes.

SO10 standard texts have a genuine delete operation, making the lifecycle reversible in TS3.

What this adds

  • create-revise-delete.flow.yaml: an 88-step create, six-version revise, display, delete, and absence-verification flow
  • create-revise-delete.trace.jsonl: the committed live TS3 recording with 93 replay actions

No runner scripts, suite manifests, or cleanup helpers are included.

End-to-end validation

  • recorded all 93 actions against TS3 client 100 in 22m31s
  • replayed every recorded action successfully with no degraded selectors
  • verified each saved version after reopening
  • deleted the text and confirmed Display returned not found
  • trace preserves ${SO10_TEXT_NAME} and does not contain the resolved sandbox name
  • git diff --check passes

Remaining before ready

  • run the full workspace fmt, clippy, and test gates when sufficient disk space is available

The PR remains draft for those repository-wide gates.

@HappyDevs1
HappyDevs1 marked this pull request as ready for review August 12, 2026 14:57
Copilot AI lite review requested due to automatic review settings August 12, 2026 14:57

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@HappyDevs1
HappyDevs1 marked this pull request as draft August 12, 2026 14:58
@HappyDevs1
HappyDevs1 marked this pull request as ready for review August 12, 2026 15:26
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.

2 participants