Skip to content

Comments

Follow-up: scan full Dependencies section for Blocked by extraction#8

Open
RyaanLari wants to merge 1 commit intocodex/update-sync-field-workflow-for-templatesfrom
codex/github-mention-update-project-field-sync-parsing-for-revise
Open

Follow-up: scan full Dependencies section for Blocked by extraction#8
RyaanLari wants to merge 1 commit intocodex/update-sync-field-workflow-for-templatesfrom
codex/github-mention-update-project-field-sync-parsing-for-revise

Conversation

@RyaanLari
Copy link
Contributor

Motivation

  • The existing parser used readSection, which collapses a markdown section to its first non-empty line, so Blocked by: lines appearing later in the Dependencies (Blocked By / Blocks) section were missed.
  • The change ensures project-field synchronization continues to work when Blocked by: appears anywhere inside the Dependencies block rather than only at the top.

Description

  • Added readSectionBlock(body, heading) to return the full section body (and ignore the GitHub sentinel) instead of just the first non-empty line.
  • Updated readBlockedByFromDependencies to use readSectionBlock(...) and iterate all lines in the Dependencies section.
  • Broadened the Blocked by matcher to accept optional markdown list markers (for example, - Blocked by: ...) to handle common template formatting.

Testing

  • Inspected the updated workflow portion with nl -ba .github/workflows/projects-field-sync.yml | sed -n '110,190p' and verified the new helper and updated usage are present (succeeded).
  • Ran rg -n "readSection\\(|readBlockedByFromDependencies|Blocked by" .github/workflows/projects-field-sync.yml to confirm references were updated (succeeded).

Codex Task

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant