Skip to content

Fix: update getBaseDir to use process.cwd() for appPath#82

Merged
harshithad0703 merged 2 commits intodevelopmentfrom
fix/dx-3140-files-not-found
Jun 26, 2025
Merged

Fix: update getBaseDir to use process.cwd() for appPath#82
harshithad0703 merged 2 commits intodevelopmentfrom
fix/dx-3140-files-not-found

Conversation

@harshithad0703
Copy link
Copy Markdown
Contributor

version bump will be made after 1 more issue fix

@harshithad0703 harshithad0703 requested a review from a team as a code owner June 25, 2025 10:24
@harshithad0703 harshithad0703 requested review from Copilot and removed request for a team June 25, 2025 10:24
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.

Pull Request Overview

This PR updates the getBaseDir function to use process.cwd() instead of deriving the application path from __dirname.

  • Updates appPath computation in getBaseDir to use process.cwd()
  • Simplifies the way the base directory is determined
Comments suppressed due to low confidence (1)

src/utils.ts:73

  • Switching to process.cwd() may lead to unexpected behavior if the current working directory is not the application root. Consider adding a comment to clarify that process.cwd() is intentionally used and state any assumptions about the runtime invocation context.
    const appPath = process.cwd();

@harshithad0703 harshithad0703 merged commit 6c1ce9a into development Jun 26, 2025
9 checks passed
@harshithad0703 harshithad0703 deleted the fix/dx-3140-files-not-found branch June 26, 2025 06:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants