Skip to content

fix: remove deprecated baseUrl from tsconfig#314

Merged
imxade merged 1 commit intoAOSSIE-Org:mainfrom
utkarsh232005:fix/tsconfig-deprecated-baseurl
Mar 15, 2026
Merged

fix: remove deprecated baseUrl from tsconfig#314
imxade merged 1 commit intoAOSSIE-Org:mainfrom
utkarsh232005:fix/tsconfig-deprecated-baseurl

Conversation

@utkarsh232005
Copy link
Copy Markdown
Contributor

@utkarsh232005 utkarsh232005 commented Mar 14, 2026 •
edited by coderabbitai bot
Loading

Addressed Issues:

N/A

Description

Remove deprecated baseUrl compiler option from tsconfig.json.

baseUrl is marked deprecated in TypeScript 6.0 and will stop functioning entirely in TypeScript 7.0 (see ts6 migration). The @/* path alias continues to resolve correctly via paths alone under moduleResolution: bundler
no functional behaviour changes.

Screenshots/Recordings:

N/A — config-only change.

Functional Verification

  • Please check off the behaviors verified with this change.

This PR is a TypeScript configuration fix. No runtime behavior is affected — all existing functionality remains unchanged.

Checklist

  • My PR addresses a single issue, fixes a single bug or makes a single improvement.
  • My code follows the project's code style and conventions
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas (N/A)
  • If applicable, I have made corresponding changes or additions to the documentation (N/A)
  • If applicable, I have made corresponding changes or additions to tests (N/A)
  • My changes generate no new warnings or errors
  • I have joined the Discord and I will share a link to this PR with the project maintainers there
  • I have read the CONTRIBUTING.md
  • Once I submit my PR, CodeRabbit AI will automatically review it and I will address CodeRabbit's comments.
  • In case of UI change I've added a demo video. (N/A)

⚠️ AI Notice: No AI was used to identify the deprecation warning via the TypeScript compiler error and to confirm the fix. The change was verified manually — removing baseUrl while retaining paths under moduleResolution: bundler is the documented migration path per https://aka.ms/ts6.

Summary by CodeRabbit

  • Chores
    • Updated TypeScript build configuration to streamline module resolution settings.

Copilot AI review requested due to automatic review settings March 14, 2026 21:22
Copilot AI reviewed Mar 14, 2026
View reviewed changes
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 14, 2026 •
edited
Loading

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 7fe4259d-462c-46f5-95bb-7217e9a0fe5f

📥 Commits

Reviewing files that changed from the base of the PR and between 71e26ef and a1026bf.

📒 Files selected for processing (1)
  • tsconfig.json
💤 Files with no reviewable changes (1)
  • tsconfig.json

Walkthrough

The baseUrl setting is removed from tsconfig.json. This eliminates an explicit base path for non-relative module imports, while path aliasing configuration remains intact. Module resolution behavior may be affected depending on other configuration rules.

Changes

Cohort / File(s) Summary
TypeScript Configuration
tsconfig.json		 Removed baseUrl setting, eliminating explicit base path for module imports.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

Typescript Lang

Poem

🐰 A path once anchored, now set free,
No baseUrl cluttering the tree,
Imports dance with lighter grace,
Configuration finds its place! ✨

🚥 Pre-merge checks | ✅ 1 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description addresses the key aspects (issue reference, technical rationale, and functional impact) but does not follow the provided template structure for functional verification and most checklist items are marked as N/A rather than properly addressed. While the description is technically sound, consider aligning more closely with the repository's template structure for consistency, even when sections are marked as N/A or not applicable.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: removing the deprecated baseUrl from tsconfig.json, which matches the actual changeset.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
📝 Coding Plan
  • Generate coding plan for human review comments

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

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

Tip

CodeRabbit can generate a title for your PR based on the changes.

Add @coderabbitai placeholder anywhere in the title of your PR and CodeRabbit will replace it with a title based on the changes in the PR. You can change the placeholder by changing the reviews.auto_title_placeholder setting.

@imxade imxade merged commit 25b2382 into AOSSIE-Org:main Mar 15, 2026
3 checks passed
@utkarsh232005 utkarsh232005 deleted the fix/tsconfig-deprecated-baseurl branch March 15, 2026 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@coderabbitai coderabbitai[bot] coderabbitai[bot] approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@utkarsh232005 @imxade