Fix issue with sliced node names in workflow menu

[cigamit]

In #760 we fixed the workflow menu for normal workflow jobs. This also fixes it for sliced jobs (like from a Federated Inventory).

[Copilot]

🟢 Approval recommended

Pull request overview

Fixes the workflow output navigation menu labeling for “sliced” workflow nodes (e.g., from Federated Inventory) where node identifiers can be blank and job names can be duplicated, ensuring the menu remains usable and navigation remains correct.

Changes:

• Add positional fallback labels (Workflow Job X/Y) when a workflow node’s identifier is blank.
• Extend the component test suite to cover sliced-node labeling, navigation, and filtering stability.

File summaries

File	Description
awx/ui/src/components/WorkflowOutputNavigation/WorkflowOutputNavigation.js	Adds positional fallback labeling for blank-identifier nodes in the workflow job selector.
awx/ui/src/components/WorkflowOutputNavigation/WorkflowOutputNavigation.test.js	Adds test coverage for sliced workflows to validate labeling and navigation behavior.

Review details

• Files reviewed: 2/2 changed files
• Comments generated: 1
• Review effort level: Lite

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

🟢 Approval recommended

Review details

• Files reviewed: 2/2 changed files
• Comments generated: 1
• Review effort level: Lite

[Copilot]

🟡 Changes recommended

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Review details

Suppressed comments (1)

awx/ui/src/components/WorkflowOutputNavigation/WorkflowOutputNavigation.js:82

• nodeLabel() does a jobNodes.findIndex(...) for blank identifiers, and nodeLabel() is also used inside filtering. For larger workflows this can turn into O(n²) work per render; precomputing positions once per render avoids repeated linear scans.

  const nodeLabel = (node) => {
    if (stringIsUUID(node.identifier)) {
      return node.summary_fields.job.name;
    }
    if (node.identifier) {

• Files reviewed: 4/4 changed files
• Comments generated: 1
• Review effort level: Lite