Skip to content

Conversation

@ultmaster
Copy link
Contributor

No description provided.

cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Bug: File Directory Change Breaks Relative Path Resolution

The working directory determination logic was changed by removing the behavior where it defaulted to the input file's directory when args.file was provided without args.cwd. Previously, relative paths within files (e.g., context, stylesheet, or other resources) were resolved relative to the input file's location. Now, the working directory always defaults to process.cwd(), which breaks existing functionality by causing relative path resolution issues for files that expect resources to be resolved relative to their own directory.

packages/poml/index.ts#L118-L124

poml/packages/poml/index.ts

Lines 118 to 124 in 4ff33e3

// Determine the working directory
let workingDirectory: string;
if (args.cwd) {
workingDirectory = path.resolve(args.cwd);
} else {
workingDirectory = process.cwd();
}

Fix in Cursor


Was this report helpful? Give feedback by reacting with 👍 or 👎

@ultmaster ultmaster merged commit 34a5531 into main Jun 27, 2025
4 checks passed
@ultmaster ultmaster deleted the example-expects branch July 9, 2025 09:25
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.

2 participants