RU-T39 Trying to fix changerawr build.

[ucswift]

Summary by CodeRabbit

Release Notes

• Chores
  • Updated internal build system and CI/CD pipeline infrastructure for improved reliability and consistency in release artifact generation.

[coderabbitai]

Walkthrough

The GitHub Actions CI/CD workflow is updated to modify JSON payload construction for release fields. The changes include converting title expressions to parenthesized form, adjusting content field quoting, and adding explicit type-casting to string-typed tag fields using tostring.

Changes

Cohort / File(s)	Summary
GitHub Actions Workflow Configuration .github/workflows/react-native-cicd.yml	Updated JSON payload construction: title expression changed from concatenation to parenthesized form; content field changed from quoted string to direct value reference; platform, buildNumber, commitSha, and buildUrl tag fields now use explicit tostring type-casting

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

• Single file with expression and syntax updates only
• No logic changes, control flow alterations, or API modifications
• Straightforward verification of type-casting and expression syntax

Possibly related PRs

• CU-8687aj3gt More ios build changes #157: Both PRs modify the same GitHub Actions workflow file and adjust how release fields are expressed and quoted in the Changerawr payload.
• RU-T39 Changelog updates #177: Both PRs update the CI workflow's Changerawr payload construction, including title formatting and tag field type-casting.

Poem

🐰 Type-casting hops with gentle care,
Parentheses dance through workflows fair,
Tostring spells make fields complete,
Our CI/CD now feels so sweet! ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.
Title Check	❓ Inconclusive	The pull request title "RU-T39 Trying to fix changerawr build" is vague and uses non-descriptive language that fails to convey meaningful information about the actual changeset. The phrase "Trying to fix" is imprecise and lacks specificity about what problem is being addressed. The term "changerawr" is unclear and unexplained, potentially being a typo or unexpanded abbreviation. While the title does reference builds and the changes are indeed in a CI/CD workflow file, the specific technical nature of the modifications—JSON payload restructuring, explicit type casting, and expression reformatting in the workflow configuration—is not captured or implied by the title. A teammate scanning the PR history would not understand what specific changes were made without reviewing the full changeset.	Consider revising the title to be more specific and descriptive. A better title might clearly identify the specific changes made, such as "Add type casting to JSON payload fields in React Native CI/CD workflow" or "Restructure JSON payload expressions in release workflow," which would give reviewers immediate clarity about the nature of the modifications without requiring them to inspect the full diff.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch develop

---

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.}

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

.github/workflows/react-native-cicd.yml (1)

421-421: Redundant tostring filters on already-stringified variables.

The variables passed via --arg (platform, buildNumber, commitSha, buildUrl) are already strings in jq. Applying | tostring is defensive but redundant—--arg guarantees string values, so these filters have no practical effect. They won't break anything, but they can be simplified to $platform, $buildNumber, etc. if you prefer clarity.

If you'd like to simplify this, you can remove the | tostring filters:

- "name": ($platform | tostring)
+ "name": $platform

Apply the same change to lines 425, 429, and 433.

Also applies to: 425-425, 429-429, 433-433

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 15e24da and 4da672c.

📒 Files selected for processing (1)

• .github/workflows/react-native-cicd.yml (1 hunks)

🔇 Additional comments (1)

.github/workflows/react-native-cicd.yml (1)

416-417: Critical bug fix: Content field now sends actual release notes instead of literal string.

Line 417 is the key fix here. The original "$notes" was a literal string in the jq filter, which would have sent the text $notes to the API instead of the actual release notes content. Changing to $notes correctly sends the variable value. Line 416's parenthesized concatenation also properly formats the title string.

[ucswift]

Approve

[github-actions]

This PR is approved.