fix: remove default value for enabled field

[feitianbubu]

避免mysql下每次重启autoMigrate都会重复执行
ALTER TABLE custom_oauth_providersMODIFY COLUMNenabled boolean default false

Summary by CodeRabbit

• Chores
  • Updated OAuth provider configuration to modify default value handling for the enabled state.

Walkthrough

The Enabled boolean field in the CustomOAuthProvider struct has its GORM default tag annotation removed while preserving the JSON serialization tag, affecting field declaration metadata.

Changes

Cohort / File(s)	Summary
GORM Tag Removal model/custom_oauth_provider.go	Removed gorm:"default:false" tag from the Enabled field, keeping the json:"enabled" tag intact.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• fix: mySQL does not support default false #1393: Removes gorm:"default:false" tags from boolean struct fields (Log.IsStream, Token fields) following the same pattern.
• fix: prevent loop auto-migrate with bool default false #1672: Removes gorm:"default:false" from exported boolean struct fields to address MySQL/TINYINT auto-migrate issues.
• fix: CrossGroupRetry default false #2570: Removes gorm:"default:false" tag from a boolean field in a model struct using an identical approach.

Suggested reviewers

• seefs001

Poem

🐰 A tag once bound, now set free,
The gorm default no longer to be,
One field cleaned, one change so neat,
The CustomOAuth provider's refactor complete! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title directly and clearly describes the main change: removing the default value from the enabled field in the CustomOAuthProvider struct.
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

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In `@model/custom_oauth_provider.go`:
- Line 14: The Enabled field in the CustomOAuthProvider struct is currently
nullable; update its GORM/DB struct tag to include a NOT NULL constraint so the
database enforces non-null booleans (i.e., change the tags on the Enabled field
in model/custom_oauth_provider.go to add `not null`), and ensure any migration
or schema generation will pick up the change so the column is created/altered
with NOT NULL to prevent NULL values.

⚠️ Potential issue | 🟡 Minor

🧩 Analysis chain

🏁 Script executed:

cat go.mod | grep -i gorm | head -5

Repository: QuantumNous/new-api

Length of output: 145

---

🏁 Script executed:

cat -n model/custom_oauth_provider.go

Repository: QuantumNous/new-api

Length of output: 7521

---

🏁 Script executed:

echo "=== Checking how CustomOAuthProvider is created ===" && \
rg -n --type=go 'CustomOAuthProvider\s*\{' -A 5 | head -50

Repository: QuantumNous/new-api

Length of output: 2222

---

🏁 Script executed:

echo "=== Checking Create operations for CustomOAuthProvider ===" && \
rg -n --type=go '\.Create\(|NewCustomOAuthProvider' --type=go -B 2 -A 3 | head -60

Repository: QuantumNous/new-api

Length of output: 2534

---

🏁 Script executed:

echo "=== Checking migrations for CustomOAuthProvider ===" && \
find . -name "*.go" -path "*/migration*" -o -path "*/migrations*" | head -10 && \
rg -n 'CustomOAuthProvider|custom_oauth_provider' --type=go | grep -i "migration\|migrate\|table" | head -20

Repository: QuantumNous/new-api

Length of output: 196

---

Add not null to the Enabled field for semantic clarity.

The field is currently nullable without a default constraint, which is imprecise for a feature flag. Although the creation code always explicitly sets Enabled, the column definition should enforce this intent with a not null constraint to prevent accidental NULL values if the code path changes:

Suggested fix

-	Enabled               bool   `json:"enabled"`                                                        // Whether this provider is enabled
+	Enabled               bool   `json:"enabled" gorm:"not null;default:false"`                          // Whether this provider is enabled

🤖 Prompt for AI Agents

In `@model/custom_oauth_provider.go` at line 14, The Enabled field in the
CustomOAuthProvider struct is currently nullable; update its GORM/DB struct tag
to include a NOT NULL constraint so the database enforces non-null booleans
(i.e., change the tags on the Enabled field in model/custom_oauth_provider.go to
add `not null`), and ensure any migration or schema generation will pick up the
change so the column is created/altered with NOT NULL to prevent NULL values.