PRODUCT_REVIEW.md

# Rev31 review and PCSWMM-only conversion decisions

## Retained from Rev31 as shared internal engineering core

- isolated OpenSWMM execution and integrity gate;
- INP parsing and deterministic result summaries;
- `.rpt` reconciliation;
- Calgary screening and report criteria;
- preliminary deterministic findings;
- report engine and audit ZIP generation;
- revision normalization;
- City-comment response matrix;
- submission readiness workflow;
- PCSWMM package validation and hashing.

## Removed from the public product surface

- generic `upload_model` entry workflow;
- generic scenario-authoring tools;
- generic SQL/query tools;
- standalone SWMM user positioning;
- Hugging Face deployment language and cloud-upload consent workflow;
- generic SWMM model-authoring skill;
- non-PCSWMM client examples.

These functions may still exist in the internal core because independent execution and reporting use them. They are not registered as PCSWMM MCP tools.

## Added for PCSWMM-only operation

- PCSWMM-specific public tool registry;
- active-project connection and identity endpoint;
- independent verification endpoint;
- consolidated active-model review;
- PCSWMM-prefixed result tools;
- PCSWMM submission/report wrappers;
- local-only launch scripts with an isolated OpenSWMM worker virtual environment;
- PCSWMM-specific orchestration skill and notebook client example.

## Deliberately deferred

- write access to the active PCSWMM model;
- automatic conduit/storage editing;
- optimization and scenario creation inside PCSWMM;
- direct extraction of proprietary GUI drawings or profiles not exposed by the SDK;
- automatic City-comment OCR.

These require explicit approval, rollback, snapshot, and audit controls before production use.