)]}'
{
  "commit": "454f8864b1830ff13f19a49dd53d91b5a201baf7",
  "tree": "d52cec3d8a26fe8f6f0883008aaef86b425e55cb",
  "parents": [
    "289525bcecca82247e0908c5ff1f213213d9c9a4"
  ],
  "author": {
    "name": "Josh Bleecher Snyder",
    "email": "josharian@gmail.com",
    "time": "Tue Jul 08 04:12:28 2025 +0000"
  },
  "committer": {
    "name": "Josh Bleecher Snyder",
    "email": "josharian@gmail.com",
    "time": "Mon Jul 07 21:14:34 2025 -0700"
  },
  "message": "sketch/loop: remove flaky TestPortMonitor_IntegrationDemo test\n\nDelete the integration test that was still flaky despite previous fixes.\n\nRoot Cause:\n- Test suffered from inherent race condition with system-wide port resources\n- After closing test ports, other processes could immediately bind to same port numbers\n- Race condition window existed between port closure and PortMonitor detection\n- Cannot be made truly hermetic because port numbers are OS-managed system resources\n\nTest Coverage Analysis:\n- Integration test added minimal unique value over existing unit tests\n- Agent.pushToOutbox already tested in agent_test.go\n- Port detection logic thoroughly tested in port_monitor_test.go\n- TCP filtering, sorting, added/removed detection all covered by unit tests\n- Core functionality has excellent test coverage without the integration test\n\nThe Fix:\n- Remove the flaky integration test entirely\n- Preserve all existing unit tests which provide reliable coverage\n- Eliminate CI/CD reliability issues caused by environmental dependencies\n- Remove maintenance burden of debugging false test failures\n\nThis removes the flaky test while maintaining comprehensive test coverage\nthrough the existing unit test suite.\n\nCo-Authored-By: sketch \u003chello@sketch.dev\u003e\nChange-ID: s8bd05f43558fc5c4k\n",
  "tree_diff": [
    {
      "type": "delete",
      "old_id": "4c67790b1f9b16f5e7fa1f995a4b61517310e1c9",
      "old_mode": 33188,
      "old_path": "loop/port_monitor_demo_test.go",
      "new_id": "0000000000000000000000000000000000000000",
      "new_mode": 0,
      "new_path": "/dev/null"
    }
  ]
}