Skip to content

Commit 3b9329e

Browse files
authored
[workflow-editor] workshop-builder: remove redundant blocker list and tabulate focus hints (#1605)
1 parent a27fe71 commit 3b9329e

2 files changed

Lines changed: 20 additions & 27 deletions

File tree

.github/workflows/workshop-builder.lock.yml

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/workflows/workshop-builder.md

Lines changed: 19 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -207,20 +207,8 @@ gh search issues --repo "$GITHUB_REPOSITORY" --state open \
207207

208208
Record:
209209

210-
- Whether an open PR with label `workshop` exists (blocks `workshop-author`)
211-
- Whether an open PR with label `workflow-editor` exists (blocks
212-
`workflow-skills-editor`)
213-
- Whether an open PR with label `side-quest` exists (blocks `side-quest`)
214-
- Whether an open PR with label `skill-activity` exists (blocks
215-
`workshop-skill-activity-author`)
216-
- Whether an open PR with label `security-side-quest` exists (blocks
217-
`security-side-quest`)
218-
- Whether an open PR with label `training-plan` exists (blocks
219-
`training-plan-research`)
220-
- Whether an open issue with label `dedup` exists (blocks `markdown-dedup`)
221-
- Whether an open PR with label `diagram-generator` exists (blocks
222-
`workshop-explanatory-diagrams`)
223-
- Whether an open PR with label `docs-linker` exists (blocks `docs-linker`)
210+
- Whether the open PRs and issues satisfy the label-based blockers in the
211+
eligibility table in section 2c
224212
- Any open issues suggesting improvements or reporting errors
225213

226214
### 2c. Derive dispatch eligibility
@@ -288,18 +276,23 @@ Select the highest-priority **eligible** workflow (most urgent first):
288276
17. `guidelines-enforcer` — daily round-robin enforcement of authoring guidelines; creates issues for violations
289277

290278
If `${{ inputs.focus }}` is provided (and not `"status"`), treat it as a hint
291-
that may shift priority toward a specific workflow (e.g. "add content" → prefer
292-
`workshop-author` or `training-plan-research`; "workflow", "tone",
293-
"duplication", or "bloat" → prefer `workflow-skills-editor`; "side quest" or
294-
"tutorial" → prefer `side-quest`; "security" → prefer `security-side-quest`;
295-
"quality", "curriculum", "rubric", "cognitive", or "scaffold" → prefer
296-
`curriculum-evaluator`; "fix sync" → prefer `workshop-sync-check`; "title",
297-
"heading", or "similar" → prefer `title-similarity-review`; "diagram" or
298-
"visual" → prefer `workshop-explanatory-diagrams`; "docs" or "link" → prefer
299-
`docs-linker`; "dedup" or "duplicate" → prefer `markdown-dedup`; "broken" or
300-
"checker" → prefer `workshop-link-checker`; "screenshot" or "image" → prefer
301-
`workshop-ui-screenshots`; "guidelines", "rules", or "enforce" → prefer
302-
`guidelines-enforcer`).
279+
that may shift priority toward a specific workflow:
280+
281+
| Focus keywords | Preferred workflow |
282+
|---|---|
283+
| `add content` | `workshop-author` or `training-plan-research` |
284+
| `workflow`, `tone`, `duplication`, `bloat` | `workflow-skills-editor` |
285+
| `side quest`, `tutorial` | `side-quest` |
286+
| `security` | `security-side-quest` |
287+
| `quality`, `curriculum`, `rubric`, `cognitive`, `scaffold` | `curriculum-evaluator` |
288+
| `fix sync` | `workshop-sync-check` |
289+
| `title`, `heading`, `similar` | `title-similarity-review` |
290+
| `diagram`, `visual` | `workshop-explanatory-diagrams` |
291+
| `docs`, `link` | `docs-linker` |
292+
| `dedup`, `duplicate` | `markdown-dedup` |
293+
| `broken`, `checker` | `workshop-link-checker` |
294+
| `screenshot`, `image` | `workshop-ui-screenshots` |
295+
| `guidelines`, `rules`, `enforce` | `guidelines-enforcer` |
303296

304297
When dispatching `workshop-author`, `training-plan-research`,
305298
`workflow-skills-editor`, `side-quest`, `security-side-quest`,

0 commit comments

Comments
 (0)