fix(compile): copy NDJSON manifest to artifact, guard conclusion.js, align title-prefix

1. Copy safe-outputs-executed.ndjson into the safe_outputs artifact
   so the Conclusion job can find noop/missing-tool/missing-data
   entries. Without this, diagnostic signal reporting was silently
   skipped (pipeline-failure reporting was unaffected).

2. Extend bash guard to also check conclusion.js exists on disk
   before running node — handles download failures gracefully.

3. Align title-prefix with gh-aw: it is a prefix prepended to the
   pipeline name (${titlePrefix} ${pipelineName}), not a template
   with placeholder substitution. Matches gh-aw missing_issue_helpers.

4. Move JobVariable import to file-level (consistency with other IR
   imports).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: jamesadevine

scripts/ado-script/src/conclusion/__tests__/index.test.ts

@@ -257,7 +257,7 @@ describe("conclusion/main", () => {
       "MyProject",
       expect.objectContaining({
         enabled: true,
-        title: "[ado-aw] Agent noop",
+        title: "[ado-aw] Agent noop feature reporter",
         workItemType: "Bug",
         areaPath: "MyProject\\Automation",
         tags: ["pipeline-failure", "automated"],

scripts/ado-script/src/conclusion/index.ts

@@ -370,21 +370,18 @@ function buildMissingDataReport(
   };
 }
 
-function renderTitle(
-  template: string | undefined,
+/**
+ * Build the work-item title from the per-tool title-prefix.
+ * Mirrors gh-aw's convention: `${titlePrefix} ${pipelineName}`.
+ * When no prefix is configured, returns undefined so the caller
+ * can fall back to the signal's built-in default title.
+ */
+function buildTitle(
+  titlePrefix: string | undefined,
   pipelineName: string,
-  signal: SignalKind,
-  defaultTitle: string,
 ): string | undefined {
-  if (!template) return undefined;
-  const signalLabel = signal.replaceAll("_", " ");
-  const rendered = template
-    .replaceAll("{pipeline_name}", pipelineName)
-    .replaceAll("{pipeline-name}", pipelineName)
-    .replaceAll("{pipeline}", pipelineName)
-    .replaceAll("{signal}", signalLabel)
-    .trim();
-  return rendered || defaultTitle;
+  if (!titlePrefix) return undefined;
+  return `${titlePrefix} ${pipelineName}`.trim();
 }
 
 function getToolConfigKey(kind: SignalKind): string {
@@ -407,11 +404,9 @@ function buildWorkItemConfig(
     // in fileSignal(). The field exists in WorkItemReportConfig for
     // callers outside conclusion.js (e.g. direct wit.ts consumers).
     enabled: true,
-    title: renderTitle(
+    title: buildTitle(
       toolConfig?.titlePrefix,
       config.pipelineName,
-      signal.kind,
-      signal.defaultTitle,
     ),
     workItemType: toolConfig?.workItemType ?? "Task",
     areaPath: toolConfig?.areaPath,

src/compile/agentic_pipeline.rs

@@ -1124,10 +1124,10 @@ fn build_conclusion_job(
     steps.push(Step::Task(download_artifact));
 
     let conclusion_script = "\
-if command -v node >/dev/null 2>&1; then\n  \
+if command -v node >/dev/null 2>&1 && [ -f /tmp/ado-aw-scripts/ado-script/conclusion.js ]; then\n  \
   node /tmp/ado-aw-scripts/ado-script/conclusion.js\n\
 else\n  \
-  echo \"##vso[task.logissue type=warning]Node.js not available; skipping conclusion reporting\"\n\
+  echo \"##vso[task.logissue type=warning]conclusion.js unavailable; skipping conclusion reporting\"\n\
 fi\n";
     let mut conclusion_step = bash("Report pipeline conclusion", conclusion_script);
     conclusion_step = conclusion_step.with_condition(Condition::Always);