Skip to content

fix: Removes console logs from ContainerTable.tsx file - #176

Merged
rodriguecyber merged 1 commit into
Open-Source-Kigali:developfrom
aryansinha1908:issue-164
Aug 29, 2026
Merged

fix: Removes console logs from ContainerTable.tsx file#176
rodriguecyber merged 1 commit into
Open-Source-Kigali:developfrom
aryansinha1908:issue-164

Conversation

@aryansinha1908

Copy link
Copy Markdown
Contributor

Summary

Removed two leftover debug console.log statements from the renderPort function in apps/web/src/components/ContainerTable.tsx. This eliminates console noise (which previously caused 120 writes per render on a 40-container host) and prevents leaking internal data structures in production.

Type of change

  • Bug fix
  • New feature
  • Refactor / cleanup
  • Docs
  • Infrastructure / CI
  • Breaking change

Areas touched

  • apps/server (NestJS)
  • apps/web (React / Vite)
  • apps/agent (Go)
  • packages/protocol (shared WebSocket contracts)
  • infrastructure / .github/workflows
  • docs

Related issues

Closes #164

How to test

  1. Run grep -rn "console.log" apps/web/src and confirm it returns no results.
  2. Run npm run lint --workspace=@docksight/web and ensure it passes.
  3. Start the dashboard (npm run dev:web), navigate to a page with containers, and confirm the browser console is quiet while the ports column renders exactly as before.

Checklist

  • Change is focused and stays within the existing modular monolith boundaries.
  • Ran the relevant checks locally (see below).
  • Updated docs under docs/ if module boundaries or the protocol changed.
  • Added an ADR under docs/decisions/ if this is a larger architectural change.
Local checks
# packages/protocol must be built first — apps/server and apps/web
# import it from its dist output.
npm run build --workspace=@docksight/protocol
npm run test  --workspace=@docksight/protocol

npm run build --workspace=@docksight/server
npm run lint  --workspace=@docksight/server
npm run test  --workspace=@docksight/server

npm run build --workspace=@docksight/web
npm run lint  --workspace=@docksight/web

Protocol changes

  • Updated packages/protocol/src and the fixtures under packages/protocol/fixtures.
  • Server, web, and agent were all updated for the new contract.
  • Noted whether older agents remain compatible.

Screenshots

Copilot AI lite review requested due to automatic review settings August 29, 2026 07:14

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR removes leftover debug logging from the web dashboard’s container table to eliminate excessive browser console noise and avoid exposing internal data shapes during normal usage.

Changes:

  • Removed two debug console.log calls from renderPort in the container table.
  • Verified the web source tree contains no remaining console.log usages under apps/web/src.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@rodriguecyber
rodriguecyber merged commit 63cc1c8 into Open-Source-Kigali:develop Aug 29, 2026
1 check passed
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.

Debug console.log calls left in the container table

3 participants