Skip to content

Retire unused activity file endpoints - #110

Open
deancochran wants to merge 12 commits into
devfrom
api-retire-unused-endpoints
Open

Retire unused activity file endpoints#110
deancochran wants to merge 12 commits into
devfrom
api-retire-unused-endpoints

Conversation

@deancochran

@deancochran deancochran commented Jul 8, 2026

Copy link
Copy Markdown
Owner

Summary

  • retire unused legacy activity-file management endpoints with zero app/non-test references:
    • activityFiles.uploadActivityFile
    • activityFiles.analyzeActivityFile
    • activityFiles.getActivityFileStatus
    • activityFiles.listActivityFiles
    • activityFiles.deleteActivityFile
  • retire unused profile helper endpoints:
    • profiles.list
    • profiles.getStats
    • profiles.updateZones
  • retire unused utility endpoints:
    • routes.update
    • social.deleteComment
    • storage.deleteFile
  • retire unused analytics/coaching endpoints:
    • analytics.predictPerformance
    • coaching.invite
    • coaching.respond
    • coaching.getCoach
  • retire unused integration helper endpoints while preserving active OAuth/connect/disconnect/sync overview/sync-now flows:
    • integrations.refreshSetupData
    • integrations.refreshToken
    • integrations.cleanupExpiredStates
    • integrations.ical.addFeed
    • integrations.ical.listFeeds
    • integrations.ical.updateFeed
    • integrations.ical.removeFeed
    • integrations.wahoo.syncEvent
    • integrations.wahoo.unsyncEvent
    • integrations.wahoo.getEventSyncStatus
    • integrations.wahoo.testSync
  • retire unused feed detail endpoint:
    • feed.getActivity
  • retire unused activity-plan import endpoints:
    • activityPlans.importFromFitTemplate
    • activityPlans.importFromZwoTemplate
  • retire unused event helper endpoints and router-local endpoint logic:
    • events.getWeekCount
    • events.linkCompletion
    • events.unlinkCompletion
    • events.reconcileHistoricalCompletions
    • events.listByWeek
    • events.getProviderSyncStatus
  • retire unused training-plan helper endpoints and endpoint-only API application use-cases:
    • trainingPlans.exists
    • trainingPlans.getFeasibilityPreview
    • trainingPlans.createFromMinimalGoal
    • trainingPlans.simulateScheduleAdjustment
    • trainingPlans.getIntensityDistribution
    • trainingPlans.getIntensityTrends
    • trainingPlans.checkHardActivitySpacing
    • trainingPlans.getWeeklyTotals
    • trainingPlans.auditTemplateHealth
    • trainingPlans.removeAppliedSchedule
    • trainingPlans.shiftAppliedSchedule
    • trainingPlans.regenerateAppliedSchedule
    • trainingPlans.autoAddPeriodization
  • preserve reusable pure/domain logic in @repo/core and remove only endpoint surfaces, router orchestration, endpoint-only API use-cases, obsolete tests/mocks, and dead DB/provider helper orchestration with no callers
  • keep current active APIs intact: signed upload/download, import/process, streams, profile get/update/getZones, route create/list/detail/delete, comments add/list, avatar upload/download, coaching getRoster, analytics season-best curve, integration OAuth state/connect/disconnect/list/overview/syncNow, feed.getFeed, event list/create/update/delete/constraints/today, training plan creation-config, template apply/list/get, active plan, insight timeline/current status/curves/weekly summary
  • final corrected endpoint-reference audit reports no remaining zero-reference procedure candidates

Verification

  • pnpm --filter @repo/api check-types
  • pnpm --filter @repo/api test -- activity-files
  • pnpm --filter @repo/api test -- profiles
  • pnpm --filter @repo/api test -- routes social storage
  • pnpm --filter @repo/api test -- analytics coaching
  • pnpm --filter @repo/api test -- integrations
  • pnpm --filter @repo/api test -- feed
  • pnpm --filter @repo/api test -- activity-plans
  • pnpm --filter @repo/api test -- events
  • pnpm --filter @repo/api test -- events training-plans
  • pnpm --filter mobile check-types
  • pnpm --filter mobile test -- plan-navigation training-plan-deeplink
  • git diff --check
  • pnpm --filter @repo/api lint (reported existing warning-only diagnostics; no errors/fixes required)
  • pre-push hook passed: biome, typecheck, test, knip

Latest commits

  • cc261a6 Retire unused event provider sync status endpoint
  • 3b3066d Retire unused training plan helper endpoints
  • 17c52bf Retire unused event helper endpoints
  • 8589d28 Remove unused activity plan context import
  • 940b6bd Remove dead activity plan import helper
  • 1877da1 Retire unused activity plan import endpoints
  • 2b106fe Retire unused feed activity endpoint
  • c48f21d Retire unused integration endpoints
  • fdd66ea Retire unused analytics and coaching endpoints

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