morluto/rea

Fork 0

mirror of https://github.com/morluto/rea.git synced 2026-11-05 06:45:22 +00:00

[GH-ISSUE #1038] [Bug] URI schemes containing digits are resolved as local artifact paths #379

New issue

Closed

opened 2026-10-15 18:24:28 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-15 18:24:28 +00:00

Owner

Originally created by @6639835 on GitHub (Oct 8, 2026).
Original GitHub issue: https://github.com/morluto/rea/issues/1038

Before submitting

I searched the open and closed issues for a related report.
I removed credentials, tokens, private paths, and sensitive target data.

Area

Evidence, artifacts, or reconstruction

Observed behavior

With an inventoried file named web3:app.js, an HTML script reference web3:app.js receives resolution_status: resolved and points to that local file. Without the file, it receives not-found instead of external.

Reproduction

Call resolveArtifactPathByContext with declaredPath: "web3:app.js", sourcePath: "index.html", context: "html-reference", and a files map containing index.html and web3:app.js in the same container.

Reproduced with the production resolver; only the import suffix changed from .js to .ts. No packaged CLI/MCP run.

Expected behavior

Recognize the URI scheme and return external; digits are valid after the first scheme character.

Impact and scope

The static HTML graph can add a local loads edge for an external scheme. This is a classification/evidence issue; no external request or filesystem escape was observed.

REA version

5.0.0; commit 2bf3663f65.

Runtime and platform

Node.js 26.10.0; macOS arm64.

Target and artifact context

JavaScript/HTML artifact inventory.

Interface

Other

Earliest failure stage

Artifact or evidence registration

Relevant output or evidence

resolution_status: resolved
resolved_path: web3:app.js

Typed outcome or recovery expected

No response

Analysis or suspected cause

No response

Additional context

Resolver source

Originally created by @6639835 on GitHub (Oct 8, 2026). Original GitHub issue: https://github.com/morluto/rea/issues/1038 ### Before submitting - [x] I searched the open and closed issues for a related report. - [x] I removed credentials, tokens, private paths, and sensitive target data. ### Area Evidence, artifacts, or reconstruction ### Observed behavior With an inventoried file named `web3:app.js`, an HTML script reference `web3:app.js` receives `resolution_status: resolved` and points to that local file. Without the file, it receives `not-found` instead of `external`. ### Reproduction Call `resolveArtifactPathByContext` with `declaredPath: "web3:app.js"`, `sourcePath: "index.html"`, `context: "html-reference"`, and a files map containing `index.html` and `web3:app.js` in the same container. Reproduced with the production resolver; only the import suffix changed from .js to .ts. No packaged CLI/MCP run. ### Expected behavior Recognize the URI scheme and return `external`; digits are valid after the first scheme character. ### Impact and scope The static HTML graph can add a local `loads` edge for an external scheme. This is a classification/evidence issue; no external request or filesystem escape was observed. ### REA version 5.0.0; commit 2bf3663f654386c9fd8c1437c49659df97bd4697. ### Runtime and platform Node.js 26.10.0; macOS arm64. ### Target and artifact context JavaScript/HTML artifact inventory. ### Interface Other ### Earliest failure stage Artifact or evidence registration ### Relevant output or evidence ```text resolution_status: resolved resolved_path: web3:app.js ``` ### Typed outcome or recovery expected _No response_ ### Analysis or suspected cause _No response_ ### Additional context [Resolver source](https://github.com/morluto/rea/blob/2bf3663f654386c9fd8c1437c49659df97bd4697/src/application/javascript/JavaScriptArtifactPathResolution.ts)

bsakaguchi

2026-10-15 18:24:28 +00:00

closed this issue
added the
triage

bug
labels

bsakaguchi referenced this issue

2026-10-16 21:13:04 +00:00

[PR #433] [MERGED] feat: add runtime reconstruction roadmap workflows #1982

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

morluto/rea#379

No description provided.

Rows
Columns