Skip to content

feat(tools): enhance map/replay managers and refine tools sidebar ui - #7

Merged
undead2146 merged 1 commit into
mainfrom
feat/map-manager
Jan 3, 2026
Merged

feat(tools): enhance map/replay managers and refine tools sidebar ui#7
undead2146 merged 1 commit into
mainfrom
feat/map-manager

Conversation

@undead2146

@undead2146 undead2146 commented Jan 3, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • Chores
    • Improved CI/CD pipeline security and maintainability through script-based token injection automation across build workflows.

✏️ Tip: You can customize this high-level summary in your review settings.

@netlify

netlify Bot commented Jan 3, 2026

Copy link
Copy Markdown

Deploy Preview for generalshub ready!

Name Link
🔨 Latest commit 061ac45
🔍 Latest deploy log https://app.netlify.com/projects/generalshub/deploys/695862cef49868000887ed45
😎 Deploy Preview https://deploy-preview-7--generalshub.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai

coderabbitai Bot commented Jan 3, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

A new PowerShell script is added to centralize token obfuscation and injection, replacing inline secret-handling logic in CI/CD workflows. The script generates a random key, XOR-obfuscates the token, and embeds hex-encoded arrays into ApiConstants.cs. Two workflows are updated to call this script instead of performing inline token operations.

Changes

Cohort / File(s) Summary
Token Injection Script
.github/scripts/inject-token.ps1
New PowerShell script that accepts a token parameter, validates ApiConstants.cs exists, generates a random 32-byte key, XOR-obfuscates the token, converts to hex byte arrays, and replaces placeholder constants in the file. Exits gracefully if token is empty.
CI Workflow Integration
.github/workflows/ci.yml
Adds three new steps across Windows and Linux build paths that invoke the token injection script with the UPLOADTHING_TOKEN secret via pwsh after project builds complete.
Release Workflow Refactoring
.github/workflows/release.yml
Replaces inline PowerShell logic for token obfuscation and file patching with a single script invocation in both Windows and Linux build paths, delegating secret injection to the reusable script helper.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 A token hops through workflows bright,
Obfuscated safe from prying sight,
XOR magic makes secrets blend,
On scripts and CI—a trusty friend! 🔐


📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 651c1df and 061ac45.

📒 Files selected for processing (3)
  • .github/scripts/inject-token.ps1
  • .github/workflows/ci.yml
  • .github/workflows/release.yml

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

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

@undead2146
undead2146 merged commit edb86c7 into main Jan 3, 2026
8 of 9 checks passed
@coderabbitai coderabbitai Bot mentioned this pull request Jan 3, 2026
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.

1 participant