Include job outputs in needs context completion

actions-languageservice/src/complete.expressions.test.ts

@@ -192,7 +192,7 @@ jobs:
     expect(result.map(x => x.label)).toEqual(["outputs", "result"]);
   });
 
-  it.failing("needs.<job_id>.outputs includes outputs", async () => {
+  it("needs.<job_id>.outputs includes outputs", async () => {
     const input = `
 on: push
 jobs:

actions-languageservice/src/context-providers/needs.ts

@@ -1,4 +1,6 @@
 import {data} from "@github/actions-expressions";
+import {isString} from "@github/actions-workflow-parser/.";
+import {Job} from "@github/actions-workflow-parser/model/workflow-template";
 import {WorkflowContext} from "../context/workflow-context";
 
 export function getNeedsContext(workflowContext: WorkflowContext): data.Dictionary {
@@ -7,21 +9,38 @@ export function getNeedsContext(workflowContext: WorkflowContext): data.Dictiona
     return d;
   }
 
-  for (const job of workflowContext.job.needs) {
+  for (const jobID of workflowContext.job.needs) {
-    d.add(job, defaultNeedsValues());
+    const job = workflowContext.template?.jobs.find(job => job.id.value === jobID);
+    d.add(jobID, needsJobContext(job));
   }
 
   return d;
 }
 
-function defaultNeedsValues(): data.Dictionary {
+function needsJobContext(job?: Job): data.Dictionary {
   // https://docs.github.com/en/actions/learn-github-actions/contexts#needs-context
   const d = new data.Dictionary();
 
   // Once job config contains outputs, this can be populated
-  d.add("outputs", new data.Null());
+  d.add("outputs", jobOutputs(job));
 
   // Can be "success", "failure", "cancelled", or "skipped"
   d.add("result", new data.Null());
   return d;
 }
+
+function jobOutputs(job?: Job): data.Dictionary {
+  const d = new data.Dictionary();
+  if (!job?.outputs) {
+    return d;
+  }
+
+  for (let i = 0; i < job.outputs.count; ++i) {
+    const output = job.outputs.get(i);
+    if (!isString(output.key)) {
+      continue;
+    }
+    d.add(output.key.value, new data.Null());
+  }
+  return d;
+}