ci: improve JIRA Playwright reporter observability

[ggazzo]

Log each decision point (search count, reuse vs create) and validate HTTP responses so API failures surface in CI logs with status and body preview. Clarify the initial log line so it is not mistaken for a successful JIRA write. Fix the GitHub permalink in the new-issue comment template (stray comma).

Made-with: Cursor

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

Further comments

Summary by CodeRabbit

• Bug Fixes
  • Improved JIRA reporter error handling and validation across API interactions
  • Enhanced error messages and logging for better troubleshooting and debugging

Task: ARCH-2103

[dionisio-bot]

Looks like this PR is not ready to merge, because of the following issues:

• This PR is missing the 'stat: QA assured' label

Please fix the issues and try again

If you have any trouble, please check the PR guidelines

[changeset-bot]

⚠️ No Changeset found

Latest commit: 521d6bb

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[coderabbitai]

Walkthrough

Enhanced HTTP error handling in the JIRA reporter by introducing a centralized validation method (ensureJiraOk) that checks response status, parses error details, and throws contextual errors. All key HTTP operations (search, label updates, comments, issue creation) now use this method. Response parsing changed from json() to text() with explicit parsing, and logging was expanded throughout.

Changes

Cohort / File(s)

Summary

JIRA Reporter Error Handling & Validation apps/meteor/reporters/jira.ts

Added shared LOG prefix constant and private ensureJiraOk() method to centralize HTTP failure handling across search, label updates, comments, and issue creation calls. Updated response parsing from response.json() to JSON.parse(await response.text()). Enhanced logging for search results and issue operations. Modified JQL sanitization regex and added validation guards for issue creation responses.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Suggested labels

type: chore

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check	✅ Passed	The title 'ci: improve JIRA Playwright reporter observability' is clearly related to the main change—adding logging and HTTP response validation to the JIRA reporter for better observability, though 'ci:' prefix is typically used for CI configuration changes rather than code improvements.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.19%. Comparing base (2356c88) to head (521d6bb).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop   #40145      +/-   ##
===========================================
- Coverage    70.23%   70.19%   -0.04%     
===========================================
  Files         3280     3280              
  Lines       116852   116852              
  Branches     20745    20681      -64     
===========================================
- Hits         82066    82022      -44     
- Misses       31502    31546      +44     
  Partials      3284     3284              

Flag	Coverage Δ
e2e	59.68% <ø> (-0.02%)	⬇️
e2e-api	46.58% <ø> (-0.01%)	⬇️
unit	71.02% <ø> (-0.07%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[ggazzo]

/jira ARCH-2083