Skip to content

Conversation

@tamuratak
Copy link
Contributor

The linkification instructions should be updated to clarify which formats are valid. [file.ts#L10] is not a valid format and should be removed; otherwise, agents tend to use it frequently.

image

Copy link
Contributor

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 linkification instructions to remove an invalid format that was causing agents to generate incorrect link syntax. The change removes [file.ts#L10] from the list of valid formats, keeping only the standard markdown link format [file.ts](file.ts#L10).

Key Changes

  • Removed [file.ts#L10] from the list of valid linkification formats in agent instructions
  • Updated the documentation to clarify that only the standard markdown link format with parentheses is supported

@tamuratak
Copy link
Contributor Author

tamuratak commented Jan 8, 2026

ping cc: @vijayupadya @dileepyavan

vijayupadya
vijayupadya previously approved these changes Jan 9, 2026
@vs-code-engineering vs-code-engineering bot added this to the January 2026 milestone Jan 9, 2026
@dmitrivMS
Copy link
Contributor

@tamuratak There test failures, could you please take a look?

    • Valid formats: file.ts or [file.ts#L10] only. Invalid: ([file.ts#L10]) or file.ts#L10
    • Valid formats: file.ts only. Invalid: ([file.ts#L10]) or file.ts#L10

Copy link
Contributor

@dmitrivMS dmitrivMS left a comment

Choose a reason for hiding this comment

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

Tests are failing

@osortega osortega assigned dmitrivMS and unassigned osortega Jan 9, 2026
@dmitrivMS dmitrivMS added this pull request to the merge queue Jan 10, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 10, 2026
@dmitrivMS dmitrivMS added this pull request to the merge queue Jan 10, 2026
Merged via the queue into microsoft:main with commit d193e5d Jan 10, 2026
12 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants