Skip to content

[WIP] Fix console logs: Link detection does not properly parse full URLs#15824

Draft
Copilot wants to merge 1 commit intomainfrom
copilot/fix-console-log-url-detection
Draft

[WIP] Fix console logs: Link detection does not properly parse full URLs#15824
Copilot wants to merge 1 commit intomainfrom
copilot/fix-console-log-url-detection

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 2, 2026

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.


This section details on the original issue you should resolve

<issue_title>Console logs: Link detection does not properly parse full URLs</issue_title>
<issue_description>image

When I hover over the URL, it excludes the port, etc.

This may have to do with colored port numbers vite spits out.</issue_description>

<agent_instructions>Just validated this still happens</agent_instructions>

Comments on the Issue (you are @copilot in this section)

@JamesNK I think a true fix here is relatively large.

The stress playground project I added recently includes some examples of URLs mixed with colors. If the ApiService is started directly then you can see terminals detects the URLs mixed with colors:
url-color-encoding

I think the log parser needs to be completely redesigned to:

  1. Parse text to tree of tokens
  2. Have a way to get the raw text from the tokens
  3. Run URL regex on raw text (and do other log parsing)
  4. Find tokens based on index of raw text (start and end of urls, time stamps, log levels, etc)
  5. Modify tree to add URL token and then nest/split text and color tokens to fit inside
  6. Render tokens to HTML

I think there is too much here to recommend the issue for a contributor.</comment_new>
<comment_new>@davidfowl
Punting this to 8.1</comment_new>

@maddymontaquila
Copy link
Copy Markdown
Contributor

@copilot try again please

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.

Console logs: Link detection does not properly parse full URLs

2 participants