Skip to content

Repair repo after move and rename#5

Merged
jcschaff merged 1 commit into
mainfrom
move-repo-fixes
Mar 27, 2025
Merged

Repair repo after move and rename#5
jcschaff merged 1 commit into
mainfrom
move-repo-fixes

Conversation

@jcschaff

Copy link
Copy Markdown
Contributor

No description provided.

@jcschaff jcschaff self-assigned this Mar 27, 2025
@jcschaff jcschaff merged commit aab936d into main Mar 27, 2025
@jcschaff jcschaff deleted the move-repo-fixes branch March 27, 2025 16:08
jcschaff added a commit that referenced this pull request Jun 5, 2026
Five stale spots in docs/workflows-architecture.md surfaced by an audit:

1. Shape B heading said "(open)" for PR #51 — it merged ages ago.
2. Shape C diagram label on the status-endpoint edges still showed only
   the hybrid read; PR #53 added the DB-only fallback when the workflow
   query fails (and only 404s when both sources are empty).
3. Shape C diagram's POLL -> DB_SIMS arrow showed an unconditional
   "write result"; PR #53 (#5 fix) made it SUCCEEDED-only.
4. Sequence diagram had the same two omissions:
   - POLL -> DBS write now in an alt/else (SUCCEEDED inserts, FAILED /
     RUN_ID_NOT_FOUND skips).
   - "loop polling" status read now in an alt/else with the workflow-
     query happy path AND the DB-fallback path (with its own
     records-vs-404 inner alt).
5. Storage-layer prose called BiosimSims "one per unique computation"
   without noting SUCCEEDED-only since PR #53.

Pure label/annotation updates; no structural workflow change.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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