Skip to content

Validate typed original-row snapshots without asserting phase completion - #283

Merged
rmcdaniel merged 2 commits into
mainfrom
prep/123-snapshot-decoder
Nov 5, 2026
Merged

Validate typed original-row snapshots without asserting phase completion#283
rmcdaniel merged 2 commits into
mainfrom
prep/123-snapshot-decoder

Bound normalized snapshot shape checks before field lookups

98e0eae
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Nov 5, 2026 in 34s

Quality Gate passed

Issues
18 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check warning on line 161 in tests/test_worker_startup_snapshot.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Split this composite assertion into separate assertions. 

See more on https://sonarcloud.io/project/issues?id=durable-workflow_sdk-python&issues=AaGtY68VFZ2Ki6HCnQAn&open=AaGtY68VFZ2Ki6HCnQAn&pullRequest=283

Check warning on line 140 in tests/test_worker_startup_snapshot.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Split this composite assertion into separate assertions. 

See more on https://sonarcloud.io/project/issues?id=durable-workflow_sdk-python&issues=AaGtY68VFZ2Ki6HCnQAm&open=AaGtY68VFZ2Ki6HCnQAm&pullRequest=283

Check warning on line 179 in tests/test_worker_startup_snapshot.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Split this composite assertion into separate assertions. 

See more on https://sonarcloud.io/project/issues?id=durable-workflow_sdk-python&issues=AaGtY68VFZ2Ki6HCnQAq&open=AaGtY68VFZ2Ki6HCnQAq&pullRequest=283

Check warning on line 196 in tests/test_worker_startup_snapshot.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Split this composite assertion into separate assertions. 

See more on https://sonarcloud.io/project/issues?id=durable-workflow_sdk-python&issues=AaGtY68VFZ2Ki6HCnQAr&open=AaGtY68VFZ2Ki6HCnQAr&pullRequest=283

Check warning on line 106 in tests/test_worker_startup_snapshot.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Split this composite assertion into separate assertions. 

See more on https://sonarcloud.io/project/issues?id=durable-workflow_sdk-python&issues=AaGtY68VFZ2Ki6HCnQAj&open=AaGtY68VFZ2Ki6HCnQAj&pullRequest=283

Check warning on line 64 in tests/test_worker_startup_snapshot.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Split this composite assertion into separate assertions. 

See more on https://sonarcloud.io/project/issues?id=durable-workflow_sdk-python&issues=AaGtY68VFZ2Ki6HCnQAb&open=AaGtY68VFZ2Ki6HCnQAb&pullRequest=283

Check warning on line 88 in tests/test_worker_startup_snapshot.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Split this composite assertion into separate assertions. 

See more on https://sonarcloud.io/project/issues?id=durable-workflow_sdk-python&issues=AaGtY68VFZ2Ki6HCnQAf&open=AaGtY68VFZ2Ki6HCnQAf&pullRequest=283

Check warning on line 178 in tests/test_worker_startup_snapshot.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Split this composite assertion into separate assertions. 

See more on https://sonarcloud.io/project/issues?id=durable-workflow_sdk-python&issues=AaGtY68VFZ2Ki6HCnQAp&open=AaGtY68VFZ2Ki6HCnQAp&pullRequest=283

Check warning on line 124 in tests/test_worker_startup_snapshot.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Split this composite assertion into separate assertions. 

See more on https://sonarcloud.io/project/issues?id=durable-workflow_sdk-python&issues=AaGtY68VFZ2Ki6HCnQAk&open=AaGtY68VFZ2Ki6HCnQAk&pullRequest=283

Check warning on line 97 in tests/test_worker_startup_snapshot.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Split this composite assertion into separate assertions. 

See more on https://sonarcloud.io/project/issues?id=durable-workflow_sdk-python&issues=AaGtY68VFZ2Ki6HCnQAh&open=AaGtY68VFZ2Ki6HCnQAh&pullRequest=283

Check warning on line 89 in tests/test_worker_startup_snapshot.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Split this composite assertion into separate assertions. 

See more on https://sonarcloud.io/project/issues?id=durable-workflow_sdk-python&issues=AaGtY68VFZ2Ki6HCnQAg&open=AaGtY68VFZ2Ki6HCnQAg&pullRequest=283

Check warning on line 98 in tests/test_worker_startup_snapshot.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Split this composite assertion into separate assertions. 

See more on https://sonarcloud.io/project/issues?id=durable-workflow_sdk-python&issues=AaGtY68VFZ2Ki6HCnQAi&open=AaGtY68VFZ2Ki6HCnQAi&pullRequest=283

Check warning on line 134 in tests/test_worker_startup_snapshot.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Split this composite assertion into separate assertions. 

See more on https://sonarcloud.io/project/issues?id=durable-workflow_sdk-python&issues=AaGtY68VFZ2Ki6HCnQAl&open=AaGtY68VFZ2Ki6HCnQAl&pullRequest=283

Check warning on line 67 in tests/test_worker_startup_snapshot.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Split this composite assertion into separate assertions. 

See more on https://sonarcloud.io/project/issues?id=durable-workflow_sdk-python&issues=AaGtY68VFZ2Ki6HCnQAd&open=AaGtY68VFZ2Ki6HCnQAd&pullRequest=283

Check warning on line 211 in tests/test_worker_startup_snapshot.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Split this composite assertion into separate assertions. 

See more on https://sonarcloud.io/project/issues?id=durable-workflow_sdk-python&issues=AaGtY68VFZ2Ki6HCnQAs&open=AaGtY68VFZ2Ki6HCnQAs&pullRequest=283

Check warning on line 77 in tests/test_worker_startup_snapshot.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Split this composite assertion into separate assertions. 

See more on https://sonarcloud.io/project/issues?id=durable-workflow_sdk-python&issues=AaGtY68VFZ2Ki6HCnQAe&open=AaGtY68VFZ2Ki6HCnQAe&pullRequest=283

Check warning on line 65 in tests/test_worker_startup_snapshot.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Split this composite assertion into separate assertions. 

See more on https://sonarcloud.io/project/issues?id=durable-workflow_sdk-python&issues=AaGtY68VFZ2Ki6HCnQAc&open=AaGtY68VFZ2Ki6HCnQAc&pullRequest=283

Check warning on line 169 in tests/test_worker_startup_snapshot.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Split this composite assertion into separate assertions. 

See more on https://sonarcloud.io/project/issues?id=durable-workflow_sdk-python&issues=AaGtY68VFZ2Ki6HCnQAo&open=AaGtY68VFZ2Ki6HCnQAo&pullRequest=283
View more details on SonarQubeCloud