Skip to content

Correct Attractor spec inconsistencies - #9

Open
btomaj wants to merge 8 commits into
strongdm:mainfrom
btomaj:main
Open

Correct Attractor spec inconsistencies#9
btomaj wants to merge 8 commits into
strongdm:mainfrom
btomaj:main

Conversation

@btomaj

@btomaj btomaj commented Mar 26, 2026

Copy link
Copy Markdown
  • Transform.apply(graph) was inconsistently also defined as Transform.transform(graph).
  • Orphan nodes were inconsistently expected to result in both error and warning.
  • run(...) in core execution loop accepted both a context and a config as the second parameter.
  • Outcome.status was inconsistently both an enum and a string.
  • Edge shape was implied to contain both target and to_node for the same purpose.
  • Context behaviour was unspecified for loop_restart=true
  • StageStatus==FAIL was inconsistently expected to both terminate and retry.
  • wait.human was inconsistently expected to return preferred_label in its Outcome

@btomaj btomaj changed the title Correct inconsistency in Transform interface Correct spec inconsistencies Mar 26, 2026
@btomaj btomaj changed the title Correct spec inconsistencies Correct Attractor spec inconsistencies Mar 26, 2026
@btomaj
btomaj force-pushed the main branch 2 times, most recently from b3061b1 to a1f0d2d Compare March 27, 2026 00:47
@jaytaylor

Copy link
Copy Markdown
Contributor

@btomaj Rather than listing what was changed, it will be more informative if you can share what effect these changes have in terms of generated functionality.

Have you performed generation activities with the main branch as well as this PR branch? What was the difference in outcome?

@btomaj

btomaj commented Jul 29, 2026

Copy link
Copy Markdown
Author

@jaytaylor I picked up each of these inconsistencies when Codex failed to converge. Repeated runs identified implementation gaps ad nauseam, which revealed conflicts or underspecifications in the spec. I didn't know the correct solution, so I guessed.

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