[OPIK-6871] [SDK] fix: ignore media:// authority URIs in local media extraction

[JetoPistola]

User description

Details

OpenClaw media://inbound/... URIs were being treated as local absolute file paths. The media: local-path regex in media.ts allowed a leading /, so media://inbound/example.jpg matched and was normalized to /inbound/example.jpg. The attachment uploader then stat'd that non-existent path and logged an ENOENT warning, dropping the attachment.

The fix adds a negative lookahead (?!\/\/) after media: so the media:// authority form is no longer captured. Legitimate local references — media:/abs/path.jpg and media:~/path.jpg — still resolve unchanged.

Resolution via $OPENCLAW_STATE_DIR (the other option in the issue) was deliberately not taken: the codebase has no existing reference to that env var, the name itself arrived through a lossy sync, and adding that coupling would silently no-op if the name were wrong. media:// URIs are never local paths, so ignoring them is the correct fix regardless. Proper inbound-media resolution can follow once an OpenClaw state-dir contract is confirmed.

Change checklist

• User facing
• Documentation updated (if needed)
• Tests added/updated (if needed)
• Breaking changes documented (if any)

Issues

• Resolves bug: media://inbound attachments are normalized to /inbound paths #115
• OPIK-6871

Testing

Added two regression tests in src/service/media.test.ts:

• media://inbound/example.jpg (exact issue repro) collects nothing
• media://host/path/to/example.jpg (authority + host segment) collects nothing

Existing positive cases (media:/tmp/image.png, file://, markdown links) still pass. Full suite: 104 passed / 1 skipped; tsc --noEmit clean.

Documentation

N/A

---

---

Generated description

Below is a concise technical summary of the changes proposed in this PR:
Ignore OpenClaw media:// authority references when detecting local media so media.ts no longer normalizes them and the attachment uploader stops treating them as files. Add regression tests for collectMediaPathsFromString and the uploader to ensure those URIs are skipped without warnings.

Topic	Details
Media URI parsing	Add a negative lookahead to MEDIA_SCHEME_LOCAL_PATH_RE so media:// authority URIs are excluded while local media:/ paths remain normalized. Modified files (2) src/service/media.test.ts src/service/media.ts Latest Contributors(2) User Commit Date danield@comet.com [OPIK-6871] [SDK] fix:... June 16, 2026 vincentkoc@ieee.org Require explicit local... March 06, 2026
Uploader guard	Add behavior verification that scheduleMediaAttachmentUploads ignores media:// authority URIs and never calls the attachments API or emits warnings. Modified files (1) src/service/attachment-uploader.test.ts Latest Contributors(2) User Commit Date danield@comet.com test(media): cover med... June 16, 2026 vincentkoc@ieee.org Require explicit local... March 06, 2026

_{Review this PR on Baz | Customize your next review}