Skip to content

[deep-report] Fix bots field merge in compiler — imported workflow bots silently dropped #29143

@github-actions

Description

@github-actions

Description

The Schema Consistency Check (discussion #29043) confirmed a compiler bug: when a workflow is imported, its bots field is extracted into MergedBots by import_field_extractor.go:262 but never applied to the compiled output in compiler_orchestrator_workflow.go:268.

Current code (broken):

// compiler_orchestrator_workflow.go:268
workflowData.Bots = c.extractBots(frontmatter)

Compare with SkipBots which correctly merges imports:

workflowData.SkipBots = c.mergeSkipBots(c.extractSkipBots(frontmatter), importsResult.MergedSkipBots)

Expected Fix

Add a mergeBots helper (mirroring mergeSkipBots) and change line 268 to:

workflowData.Bots = c.mergeBots(c.extractBots(frontmatter), importsResult.MergedBots)

Expected Impact

Shared workflows that define bots: [dependabot[bot]] at the top level will now be correctly merged when imported. This fixes silent bot-rejection bugs in importing workflows.

Suggested Agent

Copilot or Claude code agent — single-file change with clear precedent in the same file.

Estimated Effort

Fast (< 30 min)

Data Source

DeepReport Intelligence Briefing — 2026-04-29 · Schema Consistency Check #29043 · Run §25117657405

Generated by DeepReport - Intelligence Gathering Agent · ● 654.3K ·

  • expires on May 1, 2026, 3:37 PM UTC

Metadata

Metadata

Type

No type
No fields configured for issues without a type.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions