chore fix build

[prakashchoudhary07]

• fix: remove lowercase repo name handling from deploy workflow
• fix: add missing environment variables for Docker container in deploy workflow
• fix: correct syntax for environment variable assignments in Docker run command

Date:

Developer Name:

---

Issue Ticket Number

Description

Documentation Updated?

• Yes
• No

Under Feature Flag

• Yes
• No

Database Changes

• Yes
• No

Breaking Changes

• Yes
• No

Development Tested?

• Yes
• No

Screenshots

Screenshot 1

Test Coverage

Screenshot 1

Additional Notes

Description by Korbit AI

What change is being made?

Update the deploy workflow to quote all Docker run arguments, including the network flag and environment variable values, ensuring robust handling of values.

Why are these changes being made?

To prevent issues from unquoted values (e.g., spaces or special characters) in Docker run flags and environment variables, and to align quoting consistently across all injected secrets and vars.

Is this description stale? Ask me to generate a new description by commenting /korbit-generate-pr-description

[coderabbitai]

Caution

Review failed

The pull request is closed.

Summary by CodeRabbit

• Chores
  • Standardized formatting of dynamic values in the deployment workflow to use quoted interpolations for container network and environment variables.
  • Ensures consistent handling of deployment parameters across environments.
  • No changes to application features or behavior visible to end-users.

Walkthrough

Updates the GitHub Actions deploy workflow to quote all docker run interpolations and environment variable values in the Deploy to EC2 step. No variables added or removed. No changes to images, commands, logic, or error handling.

Changes

Cohort / File(s)	Summary of Changes
CI/CD workflow quoting updates /.github/workflows/deploy.yml	Quoted docker run arguments: --network and all -e environment values (e.g., PORT, DISCORD_PUBLIC_KEY, BOT_TOKEN, GUILD_ID, QUEUE_NAME, QUEUE_URL, ENV, RDS_BASE_API_URL, VERIFICATION_SITE_URL, MAIN_SITE_URL, BOT_PRIVATE_KEY). No changes to variable set or final image reference.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

• fix: update deploy workflow to use secrets for BOT_PRIVATE_KEY #124: Alters BOT_PRIVATE_KEY sourcing in the same workflow; this PR adjusts quoting for that and other envs.
• fix: update deploy workflow to use lowercase repo name and include chore-fix-build branch #129: Modifies the same Deploy to EC2 step for image/container naming; complements these quoting changes.

Suggested reviewers

• iamitprakash

Poem

A rabbit taps the deploy drum, hop-hop—
Quotes wrapped snug so vars won’t flop.
Docker hums, the envs aligned,
Networks stringed, neatly defined.
With tidy shells and steady beat,
Our carrots ship—release complete! 🥕🚀

✨ Finishing touches

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch chore-fix-build

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between a5a1380 and 81943cf.

📒 Files selected for processing (1)

• .github/workflows/deploy.yml (1 hunks)

---

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

[korbit-ai]

I've completed my review and didn't find any issues... but I did find this shark.

                (`.
                 \ `.
                  )  `._..---._
\`.       __...---`         o  )
 \ `._,--'           ,    ___,'
  ) ,-._          \  )   _,-'
 /,'    ``--.._____\/--''

Check out our docs on how you can make Korbit work best for you and your team.

Loving Korbit!? Share us on LinkedIn Reddit and X