Skip to content

feat: ground six-pillar diagnosis in scoped discovery evidence - #235

Merged
Atom-oh merged 6 commits into
chore/web-node-refresh-20261104from
feat/service-discovery-evidence-20261105
Nov 5, 2026
Merged

Atom-oh merged 6 commits into
chore/web-node-refresh-20261104from
feat/service-discovery-evidence-20261105

Conversation

@Atom-oh

@Atom-oh Atom-oh commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor

Instance-to-database relationships and right-sizing recommendations currently lack enough service-discovery and measured utilization evidence. This change adds a scoped evidence index, six pillar specialists and a maximum two-pass analysis loop, with report facts rendered from admitted evidence.

  • Preserve same-trace resource pairs and distinguish configured paths, DNS resolution, partial Network Flow Monitor correlation and observed application calls. Add gated reads of existing Resolver logs, NFM monitors and RDS Proxy targets.
  • Qualify 14-day EC2 memory samples and Compute Optimizer recommendations before sizing advice. Use verified registered member roles without falling back to host credentials.
  • Bound inventory, graph traversal, query budgets and local observation caches. Keep ambiguous ownership and missing coverage explicit, and redact addresses and identifiers at the model boundary.

The new collection flags default off. The feature performs diagnosis and remediation proposals only; it does not change workloads, install instrumentation or provision Neptune/ElastiCache.

This PR is stacked on the runtime/dependency refresh in #227. Once that prerequisite lands, retarget this PR to dev and run its required review and CI checks against the resulting scope.

Validation on the rebased branch: worker suite 2,043 passed, 108 skipped; related web regressions 318 passed; Terraform format and validation passed, with 53 provider-mock tests passed. The skipped worker fixtures are not release acceptance. Earlier implementation verification also included the full web suite/build and required PostgreSQL CI suites; CI must verify the final integration scope. Live AWS collection and deployment acceptance remain pending.

Atom-oh and others added 6 commits November 5, 2026 02:33
…dence-20261105

Resolves the worker Lambda environment conflict by keeping both gated
maps: diagnosis_evidence_env_map (this branch) and cts_env_map (connector
target sync from #231). Terraform tests 56 passed; worker, agent and the
branch's web suites pass; tsc reports no errors beyond dev's.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@Atom-oh
Atom-oh merged commit b81752c into chore/web-node-refresh-20261104 Nov 5, 2026
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