fix(cli): force HTTP protocol for SuperTokens connection URI

[zhravan]

Issue

Link to related issue(s):
N/A - Internal bug fix

---

Description

Fixed the CLI installation configuration to use HTTP protocol for the SuperTokens authentication service connection instead of HTTPS. This resolves connection issues during installation where the SuperTokens service expects HTTP protocol.

Changes:

• Modified SUPERTOKENS_CONNECTION_URI to force HTTP protocol by replacing any HTTPS references
• Bumped CLI version to 0.1.9
• Minor formatting cleanup in docker-compose.yml (removed trailing whitespace)

---

Scope of Change

Select all applicable areas impacted by this PR:

• View (UI/UX)
• API
• CLI
• Infra / Deployment
• Docs
• Other (specify): ________

---

Screenshot / Video / GIF (if applicable)

N/A - Backend configuration change

---

Related PRs (if any)

None

---

Additional Notes for Reviewers (optional)

This fix ensures that the SuperTokens authentication service connection uses the correct HTTP protocol during CLI installation, preventing connection failures in default setups.

---

Developer Checklist

To be completed by the developer who raised the PR.

• Add valid/relevant title for the PR
• Self-review done
• Manual dev testing done
• No secrets exposed
• No merge conflicts
• Docs added/updated (if applicable)
• Removed debug prints / secrets / sensitive data
• Unit / Integration tests passing
• Follows all standards defined in Nixopus Docs

---

Reviewer Checklist

To be completed by the reviewer before merge.

• Peer review done
• No console.logs / fmt.prints left
• No secrets exposed
• If any DB migrations, migration changes are verified
• Verified release changes are production-ready

Summary by CodeRabbit

• Bug Fixes

  • Installation now configures the authentication service connection to use HTTP instead of HTTPS, improving reliability in default setups.

• Chores

  • Bumped CLI version to 0.1.9.

• Style

  • Minor formatting cleanup in docker-compose configuration.

[coderabbitai]

Walkthrough

Updates the install command to rewrite SUPERTOKENS_CONNECTION_URI by replacing "https" with "http". Bumps CLI version from 0.1.8 to 0.1.9 in pyproject. Performs a minor whitespace cleanup in docker-compose.yml. No other logic, dependency, or structural changes.

Changes

Cohort / File(s)	Summary
Install command env var rewrite cli/app/commands/install/run.py	Adjusts SUPERTOKENS_CONNECTION_URI construction to force protocol downgrade by applying .replace("https", "http") to the URI.
CLI packaging version bump cli/pyproject.toml	Increments version from 0.1.8 to 0.1.9; no other config changes.
Compose formatting tidy docker-compose.yml	Removes trailing spaces in a build arg line; no behavioral impact.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• hotfix(conf): add cli config for supertkns #481 — Also modifies SUPERTOKENS_CONNECTION_URI construction, focusing on port/config used in the URI.
• fix: env config for psql setup with supertokens #483 — Adjusts SUPERTOKENS_CONNECTION_URI handling by stripping the "/api" path.

Suggested reviewers

• raghavyuva

Poem

Hop-hop! I tweak a string, not grand,
Turn https to http on command.
Version bumps a tiny hop,
Compose cleans whitespace—mop, mop, mop.
Carrot-stamped, I merge with glee,
A tidy trail for devs to see. 🥕🐇

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

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.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The title succinctly describes the main change in the pull request by indicating that the CLI will now force the HTTP protocol for the SuperTokens connection URI, which aligns exactly with the updated logic in run.py to replace “https” with “http”.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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

[zhravan]

@coderabbitai: add description and update title

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

cli/app/commands/install/run.py (1)

464-464: Refactor SUPERTOKENS_CONNECTION_URI to avoid brittle string replace

• Define a dedicated connection_protocol = "http" and use it to build the URI instead of .replace("https", "http").
• Add a comment explaining that SuperTokens Core listens over HTTP in the local Docker setup and that HTTPS must be configured explicitly for production overrides.

[cli/app/commands/install/run.py:464]

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ade48f0 and 1b8a4a5.

📒 Files selected for processing (3)

• cli/app/commands/install/run.py (1 hunks)
• cli/pyproject.toml (1 hunks)
• docker-compose.yml (1 hunks)

🔇 Additional comments (2)

docker-compose.yml (1)

55-55: LGTM! Formatting cleanup.

The trailing whitespace removal is a good housekeeping change with no functional impact.

cli/pyproject.toml (1)

3-3: LGTM! Standard version bump.

The version increment from 0.1.8 to 0.1.9 appropriately reflects the changes in this PR.

[coderabbitai]

Tip

For best results, initiate chat on the files or code changes.

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Fetch the PR details including files changed
gh pr view 487 --json title,body,files

echo "---DIFF---"
# Get the diff for the PR
gh pr diff 487

Length of output: 3893

---

🏁 Script executed:

#!/bin/bash
# Update the PR title
gh pr edit 487 --title "fix(cli): force HTTP protocol for SuperTokens connection URI"

# Update the PR description
gh pr edit 487 --body "#### Issue
_Link to related issue(s):_  
N/A - Internal bug fix

---

#### Description
Fixed the CLI installation configuration to use HTTP protocol for the SuperTokens authentication service connection instead of HTTPS. This resolves connection issues during installation where the SuperTokens service expects HTTP protocol.

**Changes:**
- Modified \`SUPERTOKENS_CONNECTION_URI\` to force HTTP protocol by replacing any HTTPS references
- Bumped CLI version to 0.1.9
- Minor formatting cleanup in docker-compose.yml (removed trailing whitespace)

---

#### Scope of Change
_Select all applicable areas impacted by this PR:_

- [ ] View (UI/UX)
- [ ] API
- [x] CLI
- [ ] Infra / Deployment
- [ ] Docs
- [ ] Other (specify): ________

---

#### Screenshot / Video / GIF (if applicable)
_N/A - Backend configuration change_

---

#### Related PRs (if any)
_None_

---

#### Additional Notes for Reviewers (optional)
This fix ensures that the SuperTokens authentication service connection uses the correct HTTP protocol during CLI installation, preventing connection failures in default setups.

---

#### Developer Checklist
_To be completed by the developer who raised the PR._

- [x] Add valid/relevant title for the PR
- [x] Self-review done  
- [x] Manual dev testing done  
- [x] No secrets exposed  
- [x] No merge conflicts  
- [ ] Docs added/updated (if applicable)  
- [x] Removed debug prints / secrets / sensitive data  
- [ ] Unit / Integration tests passing  
- [x] Follows all standards defined in **Nixopus Docs**

---

#### Reviewer Checklist
_To be completed by the reviewer before merge._

- [ ] Peer review done  
- [ ] No console.logs / fmt.prints left  
- [ ] No secrets exposed  
- [ ] If any DB migrations, migration changes are verified
- [ ] Verified release changes are production-ready



<!-- This is an auto-generated comment: release notes by coderabbit.ai -->

## Summary by CodeRabbit

- Bug Fixes
  - Installation now configures the authentication service connection to use HTTP instead of HTTPS, improving reliability in default setups.

- Chores
  - Bumped CLI version to 0.1.9.

- Style
  - Minor formatting cleanup in docker-compose configuration.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->"

echo "PR updated successfully!"

Length of output: 118

---

@zhravan: I've updated the PR title and description for you! ✨

Updated title:
fix(cli): force HTTP protocol for SuperTokens connection URI

Description highlights:

• Added clear explanation of the bug fix (forcing HTTP protocol for SuperTokens connection)
• Listed all changes including version bump and formatting cleanup
• Marked CLI scope as impacted
• Filled in the developer checklist based on the nature of changes
• Retained the CodeRabbit summary at the bottom

The PR now has a descriptive conventional commit-style title and a complete description that explains what was fixed and why. You can view the changes here. 🐰