Skip to content

Code Health: raise TypeScript source files to 9.5 #206

Description

@woksin

Raise Code Health of the hand-written TypeScript source in Arc.TypeScript to 9.5 or higher (new files 10), with no behavior change and no public API change. Scores measured on origin/main 4ee6683 (hotspots by 90-day change count). Generated code is out of scope.

Work worst first; characterization specs before each refactor; one behavior-preserving refactor per commit; ship as minor or patch PRs through the repository train. Anything that would break public API is split into a next-major issue instead.

Files below 9.5

  • Source/Drizzle/PostgreSQLObservationManager.ts (4.55)
  • Source/Tools/ProxyGenerator/SourceTypeResolver.ts (6.48)
  • Source/Core/queries/modelBound/compileQueries.ts (7.21)
  • Source/Tools/ProxyGenerator/renderSource.ts (7.44)
  • Source/MongoDB/MongoReadModels.ts (7.51)
  • Source/Core/queries/observable/ObservableQuerySession.ts (7.69)
  • Source/Core/commands/executeCommandOperation.ts (7.85)
  • Source/Tools/ProxyGenerator/renderArtifactMetadata.ts (7.90)
  • Source/Core/http/handleRequest.ts (7.94)
  • Source/Core/ArcApplication.ts (7.96)
  • Source/Tools/ProxyGenerator/analyzeSource.ts (8.12)
  • Source/Core/http/createRouteTable.ts (8.37)
  • Source/Core/dependencyInjection/ServiceRegistry.ts (8.41)
  • Source/Drizzle/DrizzleReadModels.ts (8.54)
  • Source/Chronicle/withChronicle.ts (8.57)
  • Source/Core/ArcServer.ts (8.63)
  • Source/Core/configuration/loadConfiguration.ts (8.67)
  • Source/Chronicle/ChronicleResponseHandler.ts (8.81)
  • Source/Core/openApi/renderOpenApi.ts (8.90)
  • Source/Core/queries/renderQuery.ts (8.91)
  • Source/Core/commands/modelBound/compileCommand.ts (8.95)
  • Source/Core/dependencyInjection/ServiceScope.ts (8.96)
  • Source/Tools/ProxyGenerator/cli.ts (9.09)
  • Source/Tools/ProxyGenerator/watchSource.ts (9.28)
  • Source/Core/commands/runCommandOperation.ts (9.43)
  • Source/Core/queries/queryOperation.ts (9.44)

Acceptance criteria

  • Every listed file reaches Code Health 9.5 or higher, or has a documented reason for being left lower.
  • Each refactor is behavior-preserving and backed by specs; lint, type-check and the existing specs pass.
  • No public API or generated-proxy output changes.

Status

State: Paused — two files delivered in #207; rest deprioritized by focus change
Branch: arc23-candidate
Scope: Code Health checklist; worst-first
Delivered: Nothing yet.
Remaining: To be confirmed.
Follow-ups: None.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions