Skip to content

Commit 900b66b

Browse files
chore(saas): remove dead ErrorTrackingService island + credits path exclusion (Stirling-Tools#6718)
## What Residual dead-code cleanup following the credits engine teardown (Stirling-Tools#6687). - **Delete the `ErrorTrackingService` dead island** (6 files): the service, `UserErrorTrackerRepository`, `UserErrorTracker`, `ProcessingErrorType`, `CreditsProperties`, and `ErrorTrackingServiceTest`. These formed a self-referential cluster with **zero external callers** once the credit machinery was removed. - **Remove `/api/v1/credits/**`** from both `excludePathPatterns` blocks in `PaygWebMvcConfig` — the credits controller no longer exists, so the exclusion is defunct. (spotless collapsed the lists to one line.) ## Verification - `./gradlew :saas:compileJava :saas:compileTestJava` → **BUILD SUCCESSFUL** - grep confirms zero dangling references to the deleted types ## Not in scope (deliberately deferred) Destructive DB drops (`user_credits`/`team_credits`/`user_subscription_plans` tables, dead `payg_shadow_charge` columns, `user_error_tracker` table) are gated behind the post-release soak (`live_ratio==1.0 ≥7d`) and tracked in a separate bundle. Co-authored-by: EthanHealy01 <80844253+EthanHealy01@users.noreply.github.com>
1 parent c3795c1 commit 900b66b

7 files changed

Lines changed: 2 additions & 1261 deletions

File tree

app/saas/src/main/java/stirling/software/saas/config/CreditsProperties.java

Lines changed: 0 additions & 75 deletions
This file was deleted.

app/saas/src/main/java/stirling/software/saas/model/ProcessingErrorType.java

Lines changed: 0 additions & 139 deletions
This file was deleted.

app/saas/src/main/java/stirling/software/saas/model/UserErrorTracker.java

Lines changed: 0 additions & 95 deletions
This file was deleted.

app/saas/src/main/java/stirling/software/saas/payg/filter/PaygWebMvcConfig.java

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -62,20 +62,12 @@ public class PaygWebMvcConfig implements WebMvcConfigurer {
6262
public void addInterceptors(InterceptorRegistry registry) {
6363
registry.addInterceptor(paygChargeInterceptor)
6464
.addPathPatterns("/api/**")
65-
.excludePathPatterns(
66-
"/api/v1/credits/**",
67-
"/api/v1/config/**",
68-
"/api/v1/info/**",
69-
"/api/v1/admin/**")
65+
.excludePathPatterns("/api/v1/config/**", "/api/v1/info/**", "/api/v1/admin/**")
7066
.order(INTERCEPTOR_ORDER);
7167

7268
registry.addInterceptor(entitlementGuard)
7369
.addPathPatterns("/api/**")
74-
.excludePathPatterns(
75-
"/api/v1/credits/**",
76-
"/api/v1/config/**",
77-
"/api/v1/info/**",
78-
"/api/v1/admin/**")
70+
.excludePathPatterns("/api/v1/config/**", "/api/v1/info/**", "/api/v1/admin/**")
7971
.order(ENTITLEMENT_GUARD_ORDER);
8072
}
8173
}

app/saas/src/main/java/stirling/software/saas/repository/UserErrorTrackerRepository.java

Lines changed: 0 additions & 41 deletions
This file was deleted.

0 commit comments

Comments
 (0)