test: MongoDB config normalization — alias resolution

[anandgupta42]

What does this PR do?

1. normalizeConfig() MongoDB aliases — packages/drivers/src/normalize.ts (11 new tests)

MongoDB driver support was added in #482 but the config normalization test file (driver-normalize.test.ts, 61 existing tests covering every other driver) had zero MongoDB coverage. This is a P0 gap: users who provide MongoDB connection config using common field names from MongoDB documentation, dbt profiles, or LLM-generated output would experience silent connection failures because aliases like uri, url, connectionString, authSource, etc. would not be resolved to their canonical snake_case names.

New coverage includes:

• Connection string aliases: uri → connection_string, url → connection_string, connectionString → connection_string
• Canonical precedence: connection_string takes priority when both canonical and alias fields are present
• Auth/replica aliases: authSource → auth_source, replicaSet → replica_set
• Boolean alias: directConnection → direct_connection
• Common aliases: username → user, dbname → database (shared with other drivers)
• Type alias: mongo resolves identically to mongodb
• Timeout aliases: connectTimeoutMS → connect_timeout, serverSelectionTimeoutMS → server_selection_timeout
• Alias priority ordering: when multiple aliases (connectionString + uri) are both present, the first-listed alias wins per the applyAliases contract

Type of change

• New feature (non-breaking change which adds functionality)

Issue for this PR

N/A — proactive test coverage for newly added MongoDB driver (#482)

How did you verify your code works?

bun test test/altimate/driver-normalize.test.ts    # 89 pass (72 existing + 11 new + 6 describe groups)

All 89 tests pass. No source code was modified — tests only.

Checklist

• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes

https://claude.ai/code/session_01UtQ9rqYTKybuvdu6bM4cTj

Summary by CodeRabbit

• Tests
  • Expanded test coverage for MongoDB driver configuration handling with comprehensive validation of configuration alias normalization and canonical field mapping. Tests verify proper handling of connection string, authentication, replica set, direct connection, credential, and timeout parameter aliases to ensure consistent configuration normalization.

[claude]

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.

_{Tip: disable this comment in your organization's Code Review settings.}

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 2678429f-1422-4a81-8bce-4fd6de852d83

📥 Commits

Reviewing files that changed from the base of the PR and between abcaa1d and 728feb2.

📒 Files selected for processing (1)

• packages/opencode/test/altimate/driver-normalize.test.ts

---

📝 Walkthrough

Walkthrough

Added test suite for normalizeConfig MongoDB aliases, validating that various input aliases (uri, url, connectionString, authSource, replicaSet, directConnection, username, dbname, connectTimeoutMS, serverSelectionTimeoutMS) normalize to their canonical forms with aliases removed.

Changes

Cohort / File(s)	Summary
MongoDB Alias Normalization Tests packages/opencode/test/altimate/driver-normalize.test.ts	New test suite asserting MongoDB-specific alias-to-canonical field mappings (uri/url → connection_string, authSource → auth_source, etc.) and validating that canonical fields take precedence while alias properties are omitted from output.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• feat: add MongoDB driver support #482: Introduces the MONGODB_ALIASES configuration and normalization logic that these new tests validate against.

Suggested labels

contributor

Poem

🐰 A tester hops through MongoDB's ways,
Aliases dancing through normalize's maze,
From uri to connection_string they spring,
Each mapping validated—what joy tests bring! 🎉

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately summarizes the main change: adding MongoDB config normalization alias resolution tests to the test suite.
Description check	✅ Passed	The description comprehensively covers all required template sections with detailed information about what changed, testing verification, and a completed checklist.
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

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch test/hourly-20260327-0211

---

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

[anandgupta42]

Consolidated into #498. Tests deduplicated and merged into a single PR.