Skip to content

refactor: remove obsolete exported dead code - #120

Merged
ttbombadil merged 1 commit into
mainfrom
chore/remove-knip-dead-code
Sep 29, 2026
Merged

ttbombadil merged 1 commit into
mainfrom
chore/remove-knip-dead-code

Conversation

@ttbombadil

Copy link
Copy Markdown
Collaborator

Summary

Removes the ten pre-existing Knip diagnostics from the repository instead of
carrying them permanently as accepted baseline debt.

The cleanup removes only unused exports/types and two obsolete deprecated
internal compatibility aliases.

Removed diagnostics

Unused exports:

  • anchorCourseContentRevision
  • isTutorPlanningTransition
  • DockerCompileSemaphore
  • getDockerCompileSemaphore

Unused exported types:

  • CourseContentManifest
  • TutorPhase
  • TutorQualityCases
  • TopicExtension

The two duplicate-export diagnostics disappear with the obsolete Sandbox
compatibility aliases.

Compatibility review

The removed symbols have no consumers in repository code, tests or
documentation.

The two Sandbox aliases were marked as deprecated internal compatibility
aliases. No supported public package surface or documented external use was
found.

Used schemas and parser logic remain intact; in particular
courseContentManifestSchema is unchanged.

Verification

  • npx knip — 0 findings, exit 0
  • npm run check — passed
  • npm run check:docs — passed
  • npm run test:tutor-quality — 216 passed
  • npm run test:unit — 2477 passed / 1 skipped
  • git diff --check — passed
  • bash -n run-tests.sh — passed
  • full ./run-tests.sh — passed
  • SonarQube Quality Gate — passed
  • Sonar violations — 0

Scope

No changes to:

@ttbombadil
ttbombadil merged commit 5e84b93 into main Sep 29, 2026
5 checks passed
@ttbombadil
ttbombadil deleted the chore/remove-knip-dead-code branch September 29, 2026 09:51
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