PHANTOM
🇮🇳 IN
Skip to content

[spec-kit][sonnet] Add Parallel state support for StepFunctions TestState API#13853

Draft
tiurin wants to merge 13 commits intomainfrom
002-sfn-teststate-parallel-implementation-sonnet-4-6
Draft

[spec-kit][sonnet] Add Parallel state support for StepFunctions TestState API#13853
tiurin wants to merge 13 commits intomainfrom
002-sfn-teststate-parallel-implementation-sonnet-4-6

Conversation

@tiurin
Copy link
Contributor

@tiurin tiurin commented Feb 26, 2026

Motivation

Contributes to DRG-513

Reused planning results from #13852 (up to and including /specify.tasks) written with Opus 4.6

Then ran specify.implement using Sonnet 4.6

Changes

Tests

Related

Prompt:

/speckit.specify Add support for Parallel state to StepFunctions TestState implementation. Relevant input validations to implement: 1) mock.result is not a valid JSON array, and
definition contains a Parallel state 2) definition contains a Parallel state and mock.result is a JSON array whose size is not equal to the number of branches in the Parallel state
prompt:
/speckit.clarify Add edge cases to functional requirements
Prompt: /speckit.plan
prompt: /speckit.tasks
prompt: /speckit.constitution update with the following rules for running localstack: 1) check that localstack is NOT running by executing `localstack status` command and checking if runtime status is "stopped" in the command output. 2) start localstack as described using localstack.dev.run module 3) check if localstack is running by executing `localstack status` command and checking if runtime status is "running" in the command output. 4) always stop localstack after running tests using `localstack stop` command. 5) Check that status is "stopped" in the output of `localstack status` before proceeding
prompt: /speckit.plan update the plan taking into account the updated constitution
prompt: /speckit.tasks update tasks taking into account the updated constitution
prompt: /speckit.analyze
Purely LLM output, not reviewed yet.

LLM stopped, asking to run snapshot tests against AWS:
```
To continue: run AWS_PROFILE=ls-sandbox TEST_TARGET=AWS_CLOUD SNAPSHOT_UPDATE=1 pytest tests/aws/services/stepfunctions/v2/test_state/ -v -k "parallel" to record AWS snapshots, then
  verify with LocalStack.
```

LLM added some implementation even before running tests against AWS.

model: Sonnet 4.6
prompt: /speckit.implement
model: sonnet-4.6

Agent stopped with the following prompt for the user:

```
To continue: run AWS_PROFILE=ls-sandbox TEST_TARGET=AWS_CLOUD SNAPSHOT_UPDATE=1 pytest tests/aws/services/stepfunctions/v2/test_state/ -v -k "parallel" to record AWS snapshots, then
  verify with LocalStack.
```

I've promted agent with:
```
continue with the suggestion
```
@tiurin tiurin added this to the Playground milestone Feb 26, 2026
@github-actions
Copy link

Test Results - Preflight, Unit

23 070 tests   21 179 ✅  5m 49s ⏱️
     1 suites   1 891 💤
     1 files         0 ❌

Results for commit ec8a109.

@github-actions
Copy link

LocalStack Community integration with Pro

    2 files      2 suites   22m 45s ⏱️
1 740 tests 1 651 ✅ 89 💤 0 ❌
1 742 runs  1 651 ✅ 91 💤 0 ❌

Results for commit ec8a109.

@github-actions
Copy link

Test Results (amd64) - Acceptance

7 tests   5 ✅  3m 6s ⏱️
1 suites  2 💤
1 files    0 ❌

Results for commit ec8a109.

@github-actions
Copy link

Test Results (amd64) - Integration, Bootstrap

    5 files      5 suites   40m 30s ⏱️
1 764 tests 1 676 ✅ 88 💤 0 ❌
1 770 runs  1 676 ✅ 94 💤 0 ❌

Results for commit ec8a109.

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