Skip to content

Fix Linear and Jira smoke issue creation - #58632

Merged
pelikhan merged 6 commits into
mainfrom
copilot/fix-github-actions-job-again
Sep 5, 2026
Merged

Fix Linear and Jira smoke issue creation#58632
pelikhan merged 6 commits into
mainfrom
copilot/fix-github-actions-job-again

Conversation

Copilot AI commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

The Smoke Issues workflow failed because Jira lacked its base URL and Linear received a project URL identifier where issueCreate requires a model UUID.

  • Linear

    • Resolve 12-character project identifiers before issue creation.
    • Continue passing UUID identifiers directly.
    projects(filter: { slugId: { eq: $slugId } }, first: 1) {
      nodes { id }
    }
  • Jira

    • Configure the target Jira base URL in the workflow’s trusted safe-output environment.
  • Coverage

    • Verify project resolution and use of the resolved UUID in the Linear mutation.

Copilot AI and others added 2 commits September 4, 2026 23:45
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing GitHub Actions job Fix Linear and Jira smoke issue creation Sep 4, 2026
Copilot AI requested a review from pelikhan September 4, 2026 23:50
Comment thread .github/workflows/smoke-issues.md Outdated
engine: copilot
safe-outputs:
env:
JIRA_BASE_URL: https://pelidehalleux.atlassian.net

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot the compiler should use this env by default.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Moved the Jira base URL into the compiler-provided safe-output environment and removed the workflow override in e7c1ab8. Added focused default-injection coverage, finalized in 21a7408.

@pelikhan
pelikhan marked this pull request as ready for review September 5, 2026 01:18
Copilot AI balanced review requested due to automatic review settings September 5, 2026 01:18

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

Project resolution is repeated per issue, UUID passthrough lacks coverage, and an unrelated schema change needs separation or justification.

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

Pull request overview

Fixes Linear and Jira safe-output issue creation in the smoke workflow.

Changes:

  • Resolves Linear project slugs to UUIDs before issue creation.
  • Configures the Jira base URL and recompiles the workflow.
  • Allows null workflow_call schema values.
File summaries
File Description
actions/setup/js/linear_create_issue.cjs Adds Linear project resolution.
actions/setup/js/linear_safe_outputs.test.cjs Tests slug resolution.
.github/workflows/smoke-issues.md Configures Jira URL.
.github/workflows/smoke-issues.lock.yml Regenerates compiled workflow.
pkg/workflow/schemas/github-workflow.json Expands workflow_call validation.
Review details
  • Files reviewed: 5/5 changed files
  • Comments generated: 3
  • Review effort level: Balanced

if (LINEAR_UUID_PATTERN.test(projectId)) {
input.projectId = projectId;
} else {
const projectData = await linearGraphQL(LINEAR_RESOLVE_PROJECT, { slugId: projectId });
Comment on lines +33 to 36
fetch
.mockResolvedValueOnce(response({ data: { projects: { nodes: [{ id: "a3f91a0b-6d71-4c58-a4bb-72b925bbebc8" }] } } }))
.mockResolvedValueOnce(response({ data: { issueCreate: { success: true, issue: { id: "id", identifier: "ENG-1", title: "Safe title" } } } }));
const handler = await createIssue({ team_id: "9cfb482a-81e3-4154-b5b9-2c805e70a02d", project_id: "810f57a7e383" });
Comment on lines +1948 to 1953
"oneOf": [
{
"type": "null"
},
{
"type": "object",
Copilot AI and others added 2 commits September 5, 2026 01:28
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
@gh-aw-bot

Copy link
Copy Markdown
Collaborator
pr-sous-chef

@copilot Please take the next forward-progress pass on this PR.

  • Copilot review still points to three concrete follow-ups: avoid repeated project resolution, add UUID passthrough coverage, and either separate or justify the unrelated workflow_call schema change.
  • Please address those review points on current HEAD, refresh the branch if needed, and then run the pr-finisher skill.
  • In your handoff, summarize which review items were fixed and call out any remaining maintainer-facing blocker explicitly.

Generated by PR Sous Chef: https://github.com/github/gh-aw/actions/runs/33936161132

Generated by 👨‍🍳 PR Sous Chef · pi · gpt54 · 23.1 AIC · ⌖ 8.64 AIC · ⊞ 9.2K ·
Comment /souschef to run again

@pelikhan
pelikhan merged commit 4764ec1 into main Sep 5, 2026
3 checks passed
@pelikhan
pelikhan deleted the copilot/fix-github-actions-job-again branch September 5, 2026 01:35
Copilot stopped work on behalf of gh-aw-bot due to an error September 5, 2026 01:35
Copilot AI requested a review from gh-aw-bot September 5, 2026 01:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants