Skip to content

[Docs] Point runtime investigations at current source and local port - #3635

Open
roomote-roomote[bot] wants to merge 1 commit into
developfrom
fix/runtime-skill-source-map-0gumj5qbdydw6
Open

roomote-roomote[bot] wants to merge 1 commit into
developfrom
fix/runtime-skill-source-map-0gumj5qbdydw6

Conversation

@roomote-roomote

Copy link
Copy Markdown
Contributor

​Opened on behalf of @daniel-lxs. Follow up by mentioning @roomote-roomote, in the web UI, or in Telegram.

Related issue

Internal maintainer-requested runtime/developer workflow correction; no issue was opened.

Why this PR exists

  • A maintainer explicitly invited this PR in the linked issue or discussion
  • I am a maintainer / this is internal Roomote work

The runtime investigation skill directs its mandatory first reads to two files that no longer exist. Following it produces failed reads before an agent reaches the actual queue and recovery owners. It also conflates the local development preview port with the proxy's internal/container port.

What changed

  • Point both queue/recovery reference lists to task-run-queue.ts and orphaned-task-runs.ts.
  • Distinguish the configurable local development default 18081 from the proxy's internal/container default 8081.

The change is limited to the checked-in skill source. It adds no policy, dependencies, runtime state or application behavior.

How it was tested

  • Baseline c97173a7: both former mandatory source paths are absent; actual source owners exist. The failed reads were directly observed while following this skill.
  • Exact head 85f022d7: all 15 unique source file/directory references resolve. The queue/recovery modules export dequeueTaskRun and getOrphanedTaskRun respectively.
  • Evaluated the real ecosystem.config.js: preview proxy defaults to 18081 and respects ROOMOTE_PREVIEW_PROXY_PORT=19081. Confirmed the internal service default remains 8081.
  • git diff --check and intact commit/push gates passed, including oxlint, residual lint, workspace fast types and Knip. Browser proof and new text-mirroring tests are not applicable to this documentation-only correction.

Related PRs

Checklist

  • The PR title follows the repo convention: [Fix], [Feat], [Improve], [Refactor], [Docs], or [Chore] followed by a user-facing description
  • This PR is small and scoped to one change
  • pnpm lint and pnpm check-types pass locally
    • Normal fast push gates passed; full aliases were not rerun for this Markdown-only diff.
  • I added tests or included a clear manual validation note above
  • I removed secrets, tokens, private keys, and customer data from code, logs, and screenshots
  • If this change should appear in the changelog, I ran pnpm changeset

No changeset: internal developer skill guidance only.

@roomote-community

roomote-community Bot commented Oct 30, 2026 •

Copy link
Copy Markdown
Contributor

No code issues found. See session

Reviewed the documentation diff against the queue/recovery source owners and preview-proxy configuration. Both replacement paths are correct; the local development default 18081, ROOMOTE_PREVIEW_PROXY_PORT override, and internal default 8081 match the source.

Reviewed 85f022d

This branch has not been deployed

No deployments
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.

1 participant