[PR #1797] [CLOSED] fix(javascript): keep asterisks inside a sourceMappingURL line comment #4822

New issue

Closed

opened 2026-10-31 00:55:24 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-31 00:55:24 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/morluto/rea/pull/1797
Author: @yylm-lm
Created: 10/11/2026
Status: ❌ Closed

Base: main ← Head: fix/source-map-asterisk

📝 Commits (2)

c21de4c fix(mcp): advertise only the collection and selector pairings the server admits
f254964 fix(javascript): keep asterisks inside a sourceMappingURL line comment

📊 Changes

4 files changed (+216 additions, -29 deletions)

View changed files

📝 src/domain/analysisView/analysisView.ts (+27 -28)
📝 src/domain/javascript/javascriptStaticAnalysisCalls.ts (+7 -1)
📝 src/domain/javascript/javascriptStaticAnalysisSourceMaps.test.ts (+22 -0)
➕ tests/boundary/mcp/analysisViewSelectorAgreement.test.ts (+160 -0)

📄 Description

Closing as a duplicate: ce845ba4 (PR #1788) already fixes #1781 and #1782, with the same line-comment terminator change and the percent-decoding work this branch did not attempt. Thanks.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/morluto/rea/pull/1797 **Author:** [@yylm-lm](https://github.com/yylm-lm) **Created:** 10/11/2026 **Status:** ❌ Closed **Base:** `main` ← **Head:** `fix/source-map-asterisk` --- ### 📝 Commits (2) - [`c21de4c`](https://github.com/morluto/rea/commit/c21de4c6ce19328031128d8b799d97d466458c61) fix(mcp): advertise only the collection and selector pairings the server admits - [`f254964`](https://github.com/morluto/rea/commit/f25496403152c92bc18816e5016c9965d8192807) fix(javascript): keep asterisks inside a sourceMappingURL line comment ### 📊 Changes **4 files changed** (+216 additions, -29 deletions) <details> <summary>View changed files</summary> 📝 `src/domain/analysisView/analysisView.ts` (+27 -28) 📝 `src/domain/javascript/javascriptStaticAnalysisCalls.ts` (+7 -1) 📝 `src/domain/javascript/javascriptStaticAnalysisSourceMaps.test.ts` (+22 -0) ➕ `tests/boundary/mcp/analysisViewSelectorAgreement.test.ts` (+160 -0) </details> ### 📄 Description Closing as a duplicate: ce845ba4 (PR #1788) already fixes #1781 and #1782, with the same line-comment terminator change and the percent-decoding work this branch did not attempt. Thanks. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi

2026-10-31 00:55:24 +00:00