Skip to content

fix: fixed vcs#1756

Open
separatrixxx wants to merge 4 commits intomasterfrom
DOCSTOOLS-5456-cli
Open

fix: fixed vcs#1756
separatrixxx wants to merge 4 commits intomasterfrom
DOCSTOOLS-5456-cli

Conversation

@separatrixxx
Copy link
Contributor

When a TOC is included with mode: merge, files are physically copied to a new location. TocService recorded the original file path as sourcePath in the merged file's metadata

This caused two bugs:

  1. Missing repo prefix. sourcePath was set to a project-relative path instead of the full VCS-relative path
  2. Explicit vcsPath overridden. If the original file had an explicit vcsPath in its frontmatter, the TocService fallback still wrote sourcePath = from=

Instead of storing from as sourcePath in the merged file's metadata, store it in a dedicated mergeFrom field

@separatrixxx separatrixxx requested a review from a team as a code owner March 11, 2026 20:13
@separatrixxx separatrixxx requested review from Yurok868 and removed request for a team March 11, 2026 20:13
@sonarqubecloud
Copy link

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.

1 participant