Skip to content

chore: remove browser folder from git as it gets built during release#913

Merged
khendrikse merged 1 commit intomainfrom
chore/remove-browser-from-git
Feb 26, 2026
Merged

chore: remove browser folder from git as it gets built during release#913
khendrikse merged 1 commit intomainfrom
chore/remove-browser-from-git

Conversation

@khendrikse
Copy link
Contributor

No description provided.

@khendrikse khendrikse requested a review from a team as a code owner February 26, 2026 12:40
@github-actions github-actions bot added the type: chore work needed to keep the product and development running smoothly label Feb 26, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 26, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between e15fc6c and 4629f53.

📒 Files selected for processing (2)
  • browser/.gitkeep
  • browser/gotrue.js
💤 Files with no reviewable changes (1)
  • browser/gotrue.js

📝 Walkthrough

Summary by CodeRabbit

  • Revert
    • Removed browser-side authentication client implementation, including session management, token refresh functionality, user administration utilities, and related error handling.

Walkthrough

The entire browser/gotrue.js file was removed from the codebase. This file contained a complete GoTrue authentication client implementation, including the GoTrue class, User and Admin classes, HTTP error handling classes, token parsing and refresh logic, session persistence with localStorage, and methods for authentication operations such as sign up, login, logout, and password recovery. The deletion removes all public exports and associated authentication and session management capabilities that were provided by this module.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive No pull request description was provided by the author, making it impossible to assess whether the description relates to the changeset. Add a description explaining why the browser folder is being removed from git and how the build process handles it during release.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly indicates the main change: removing the browser folder from git because it's built during release, which aligns with the file deletion in the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/remove-browser-from-git

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

@khendrikse khendrikse merged commit ed35ea4 into main Feb 26, 2026
6 checks passed
@khendrikse khendrikse deleted the chore/remove-browser-from-git branch February 26, 2026 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: chore work needed to keep the product and development running smoothly

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants