Skip to content

Feature: Add tag/source browser for configured connections#231

Merged
smithaldon1 merged 4 commits into
mainfrom
feat/issue-217-tag-source-browser
May 26, 2026
Merged

Feature: Add tag/source browser for configured connections#231
smithaldon1 merged 4 commits into
mainfrom
feat/issue-217-tag-source-browser

Conversation

@smithaldon1
Copy link
Copy Markdown
Member

Summary

Closes #217.

  • Adds a read-only /tag-source-browser Protocol Operations page for configured OPC-UA nodes, MQTT topics, and BACnet objects.
  • Groups source entries by protocol and connection, with filters for protocol, mapping state, mapped area/asset, quality/status, and search text.
  • Links mapped source rows to normalized FactoryEvent API context when matching event data is available, without exposing endpoints, secrets, certificates, or write controls.

Test Evidence

  • cd apps/web && npm test -> 18 passed
  • cd apps/web && npm run lint -> passed
  • cd apps/web && npm run typecheck -> passed
  • cd apps/web && npm run build -> passed
  • cd apps/web && PLAYWRIGHT_API_PORT=8001 PLAYWRIGHT_WEB_PORT=3001 PLAYWRIGHT_BASE_URL=http://127.0.0.1:3001 npm run test:e2e -> first sandbox run failed to bind 127.0.0.1:8001; reran with approved local port access -> 1 passed
  • make test -> 266 passed
  • make lint -> passed
  • make typecheck -> passed
  • make docs -> passed; docs are Markdown-only and no docs checker is configured yet
  • git diff --check -> passed

Docs Impact

  • Updated apps/web/README.md with the new route and read-only behavior.
  • Updated docs/LEARNING_LOG.md with what changed, how the derived source browser works, and the next learning step.

Follow-up Work

  • Replace derived mapping state with explicit mapping records when the dedicated mapping API lands.
  • Add richer invalid/stale reason metadata once connector diagnostics return per-source mapping validation details.

Copy link
Copy Markdown
Member Author

@smithaldon1 smithaldon1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed and approved for merge

@smithaldon1 smithaldon1 merged commit 21a77d2 into main May 26, 2026
3 checks passed
@smithaldon1 smithaldon1 deleted the feat/issue-217-tag-source-browser branch May 26, 2026 18:55
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.

11. Feature: Add tag/source browser for configured connections

1 participant