fix(shared/apm.md): drop || 'all' fallback so gh-aw substitutes target (#1185)

[danielmeppiel]

Closes #1185.

Problem

#1184 introduced a defensive || 'all' fallback on the target import:

target: ${{ github.aw.import-inputs.target || 'all' }}

That broke gh-aw's substitution. The importInputsExprRegex in pkg/parser/import_field_extractor.go only matches bare ${{ github.aw.import-inputs.X }}. Anything inside the brackets (like || 'all') makes the regex skip substitution entirely, so the unresolved expression was emitted verbatim into the consuming lock workflow. At runtime with.target was empty, which apm-action then interpreted as all -- silently ignoring whatever the consumer passed via with: target: copilot.

Fix

Drop the fallback. The schema default default: all declared on the target input is already applied by gh-aw's applyImportSchemaDefaultsFromFrontmatter before substitution, so omitted-input behaviour is preserved without the belt-and-suspenders.

target: ${{ github.aw.import-inputs.target }}

Validation

Recompiled with gh aw compile pinned to v0.71.5 (matching the compiler_version stamped in current lock metadata). Diff is exactly 3 files / 3 lines:

• shared/apm.md -- 1 line, drops the fallback
• pr-review-panel.lock.yml -- gh-aw substituted the schema default in place; line now reads literal target: all
• triage-panel.lock.yml -- same

Consumers who pass with: target: copilot will now get target: copilot substituted into their lock workflow.

Follow-up (not in this PR)

It would be nice for gh-aw upstream to make importInputsExprRegex tolerant of GHA-idiom || fallbacks, since this is a footgun. Worth filing in githubnext/gh-aw.

[Copilot]

Pull request overview

Fixes gh-aw import-input substitution for the shared APM workflow by removing a non-bare expression fallback that prevented compile-time replacement of target, ensuring consumer-provided with: target: ... values flow through correctly.

Changes:

• Remove the || 'all' fallback from target: ${{ github.aw.import-inputs.target ... }} in shared/apm.md so gh-aw can substitute the value.
• Regenerate lock workflows so the default schema value compiles to a literal target: all.
• Add a changelog entry describing the fix.

Show a summary per file

File	Description
CHANGELOG.md	Adds an Unreleased “Fixed” entry for the gh-aw substitution compatibility fix.
.github/workflows/shared/apm.md	Updates the apm-action invocation to use a bare github.aw.import-inputs.target expression for gh-aw substitution.
.github/workflows/pr-review-panel.lock.yml	Regenerated compiled workflow now contains literal target: all (schema default applied).
.github/workflows/triage-panel.lock.yml	Regenerated compiled workflow now contains literal target: all (schema default applied).

Copilot's findings

• Files reviewed: 4/4 changed files
• Comments generated: 1