Skip to content

Comments

refactor(importer): remove unused code from importer package#186

Merged
javi11 merged 1 commit intomainfrom
refactor/remove-unused-importer-code
Dec 31, 2025
Merged

refactor(importer): remove unused code from importer package#186
javi11 merged 1 commit intomainfrom
refactor/remove-unused-importer-code

Conversation

@javi11
Copy link
Owner

@javi11 javi11 commented Dec 30, 2025

Summary

  • Remove unused error constants (ErrNoRetryable, ErrNoVideoFiles) from both internal/errors/errors.go and internal/importer/errors.go
  • Remove unused service methods: Database(), GetStats(), GetWorkerCount(), UpdateWorkerCount() and ServiceStats struct
  • Remove unused queue manager method: GetWorkerCount()
  • Remove unused interface methods from QueueManager, QueueOperations, and ImportService interfaces
  • Remove unused archive function: FormatInt()
  • Remove unused file extension functions: AllExtensions(), AllExtensionsWith(), AllPossibleExtensions(), WhatIsMostLikelyExtension(), sniffTextOrNZB(), isLikelyUTF8()
  • Remove unused imports: bufio, os, mimetype, strconv

Test plan

  • Verified all removed code was unused via comprehensive grep searches
  • go build ./... passes
  • go test ./internal/importer/... passes

🤖 Generated with Claude Code

@javi11 @claude
refactor(importer): remove unused code from importer package
41ad627 
Remove dead code that was never called anywhere in the codebase:

- Remove unused error constants: ErrNoRetryable, ErrNoVideoFiles
- Remove unused service methods: Database(), GetStats(), GetWorkerCount(),
  UpdateWorkerCount() and ServiceStats struct
- Remove unused queue manager method: GetWorkerCount()
- Remove unused interface methods from QueueManager, QueueOperations,
  and ImportService interfaces
- Remove unused archive function: FormatInt()
- Remove unused file extension functions: AllExtensions(),
  AllExtensionsWith(), AllPossibleExtensions(), WhatIsMostLikelyExtension(),
  sniffTextOrNZB(), isLikelyUTF8()
- Remove unused imports: bufio, os, mimetype, strconv

All removed code was verified as unused through comprehensive grep
searches across the entire codebase.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@javi11 javi11 merged commit e6e6f55 into main Dec 31, 2025
1 check passed
drondeseries pushed a commit to drondeseries/altmount that referenced this pull request Jan 2, 2026
@javi11 @claude @drondeseries
refactor(importer): remove unused code from importer package (javi11#186
2aced15 
)

Remove dead code that was never called anywhere in the codebase:

- Remove unused error constants: ErrNoRetryable, ErrNoVideoFiles
- Remove unused service methods: Database(), GetStats(), GetWorkerCount(),
  UpdateWorkerCount() and ServiceStats struct
- Remove unused queue manager method: GetWorkerCount()
- Remove unused interface methods from QueueManager, QueueOperations,
  and ImportService interfaces
- Remove unused archive function: FormatInt()
- Remove unused file extension functions: AllExtensions(),
  AllExtensionsWith(), AllPossibleExtensions(), WhatIsMostLikelyExtension(),
  sniffTextOrNZB(), isLikelyUTF8()
- Remove unused imports: bufio, os, mimetype, strconv

All removed code was verified as unused through comprehensive grep
searches across the entire codebase.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
drondeseries pushed a commit to drondeseries/altmount that referenced this pull request Jan 2, 2026
@javi11 @claude @drondeseries
refactor(importer): remove unused code from importer package (javi11#186
903c1d4 
)

Remove dead code that was never called anywhere in the codebase:

- Remove unused error constants: ErrNoRetryable, ErrNoVideoFiles
- Remove unused service methods: Database(), GetStats(), GetWorkerCount(),
  UpdateWorkerCount() and ServiceStats struct
- Remove unused queue manager method: GetWorkerCount()
- Remove unused interface methods from QueueManager, QueueOperations,
  and ImportService interfaces
- Remove unused archive function: FormatInt()
- Remove unused file extension functions: AllExtensions(),
  AllExtensionsWith(), AllPossibleExtensions(), WhatIsMostLikelyExtension(),
  sniffTextOrNZB(), isLikelyUTF8()
- Remove unused imports: bufio, os, mimetype, strconv

All removed code was verified as unused through comprehensive grep
searches across the entire codebase.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
drondeseries pushed a commit to drondeseries/altmount that referenced this pull request Jan 2, 2026
@javi11 @claude @drondeseries
refactor(importer): remove unused code from importer package (javi11#186
e6f8924 
)

Remove dead code that was never called anywhere in the codebase:

- Remove unused error constants: ErrNoRetryable, ErrNoVideoFiles
- Remove unused service methods: Database(), GetStats(), GetWorkerCount(),
  UpdateWorkerCount() and ServiceStats struct
- Remove unused queue manager method: GetWorkerCount()
- Remove unused interface methods from QueueManager, QueueOperations,
  and ImportService interfaces
- Remove unused archive function: FormatInt()
- Remove unused file extension functions: AllExtensions(),
  AllExtensionsWith(), AllPossibleExtensions(), WhatIsMostLikelyExtension(),
  sniffTextOrNZB(), isLikelyUTF8()
- Remove unused imports: bufio, os, mimetype, strconv

All removed code was verified as unused through comprehensive grep
searches across the entire codebase.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
drondeseries pushed a commit to drondeseries/altmount that referenced this pull request Jan 2, 2026
@javi11 @claude @drondeseries
refactor(importer): remove unused code from importer package (javi11#186
6a3f3af 
)

Remove dead code that was never called anywhere in the codebase:

- Remove unused error constants: ErrNoRetryable, ErrNoVideoFiles
- Remove unused service methods: Database(), GetStats(), GetWorkerCount(),
  UpdateWorkerCount() and ServiceStats struct
- Remove unused queue manager method: GetWorkerCount()
- Remove unused interface methods from QueueManager, QueueOperations,
  and ImportService interfaces
- Remove unused archive function: FormatInt()
- Remove unused file extension functions: AllExtensions(),
  AllExtensionsWith(), AllPossibleExtensions(), WhatIsMostLikelyExtension(),
  sniffTextOrNZB(), isLikelyUTF8()
- Remove unused imports: bufio, os, mimetype, strconv

All removed code was verified as unused through comprehensive grep
searches across the entire codebase.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
drondeseries pushed a commit to drondeseries/altmount that referenced this pull request Jan 2, 2026
@javi11 @claude @drondeseries
refactor(importer): remove unused code from importer package (javi11#186
cbe8fdf 
)

Remove dead code that was never called anywhere in the codebase:

- Remove unused error constants: ErrNoRetryable, ErrNoVideoFiles
- Remove unused service methods: Database(), GetStats(), GetWorkerCount(),
  UpdateWorkerCount() and ServiceStats struct
- Remove unused queue manager method: GetWorkerCount()
- Remove unused interface methods from QueueManager, QueueOperations,
  and ImportService interfaces
- Remove unused archive function: FormatInt()
- Remove unused file extension functions: AllExtensions(),
  AllExtensionsWith(), AllPossibleExtensions(), WhatIsMostLikelyExtension(),
  sniffTextOrNZB(), isLikelyUTF8()
- Remove unused imports: bufio, os, mimetype, strconv

All removed code was verified as unused through comprehensive grep
searches across the entire codebase.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
drondeseries pushed a commit to drondeseries/altmount that referenced this pull request Jan 2, 2026
@javi11 @claude @drondeseries
refactor(importer): remove unused code from importer package (javi11#186
5bb026f 
)

Remove dead code that was never called anywhere in the codebase:

- Remove unused error constants: ErrNoRetryable, ErrNoVideoFiles
- Remove unused service methods: Database(), GetStats(), GetWorkerCount(),
  UpdateWorkerCount() and ServiceStats struct
- Remove unused queue manager method: GetWorkerCount()
- Remove unused interface methods from QueueManager, QueueOperations,
  and ImportService interfaces
- Remove unused archive function: FormatInt()
- Remove unused file extension functions: AllExtensions(),
  AllExtensionsWith(), AllPossibleExtensions(), WhatIsMostLikelyExtension(),
  sniffTextOrNZB(), isLikelyUTF8()
- Remove unused imports: bufio, os, mimetype, strconv

All removed code was verified as unused through comprehensive grep
searches across the entire codebase.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
drondeseries pushed a commit to drondeseries/altmount that referenced this pull request Jan 2, 2026
@javi11 @claude @drondeseries
refactor(importer): remove unused code from importer package (javi11#186
32a5b2e 
)

Remove dead code that was never called anywhere in the codebase:

- Remove unused error constants: ErrNoRetryable, ErrNoVideoFiles
- Remove unused service methods: Database(), GetStats(), GetWorkerCount(),
  UpdateWorkerCount() and ServiceStats struct
- Remove unused queue manager method: GetWorkerCount()
- Remove unused interface methods from QueueManager, QueueOperations,
  and ImportService interfaces
- Remove unused archive function: FormatInt()
- Remove unused file extension functions: AllExtensions(),
  AllExtensionsWith(), AllPossibleExtensions(), WhatIsMostLikelyExtension(),
  sniffTextOrNZB(), isLikelyUTF8()
- Remove unused imports: bufio, os, mimetype, strconv

All removed code was verified as unused through comprehensive grep
searches across the entire codebase.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
drondeseries pushed a commit to drondeseries/altmount that referenced this pull request Jan 2, 2026
@javi11 @claude @drondeseries
refactor(importer): remove unused code from importer package (javi11#186
4e63947 
)

Remove dead code that was never called anywhere in the codebase:

- Remove unused error constants: ErrNoRetryable, ErrNoVideoFiles
- Remove unused service methods: Database(), GetStats(), GetWorkerCount(),
  UpdateWorkerCount() and ServiceStats struct
- Remove unused queue manager method: GetWorkerCount()
- Remove unused interface methods from QueueManager, QueueOperations,
  and ImportService interfaces
- Remove unused archive function: FormatInt()
- Remove unused file extension functions: AllExtensions(),
  AllExtensionsWith(), AllPossibleExtensions(), WhatIsMostLikelyExtension(),
  sniffTextOrNZB(), isLikelyUTF8()
- Remove unused imports: bufio, os, mimetype, strconv

All removed code was verified as unused through comprehensive grep
searches across the entire codebase.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
drondeseries pushed a commit to drondeseries/altmount that referenced this pull request Jan 2, 2026
@javi11 @claude @drondeseries
refactor(importer): remove unused code from importer package (javi11#186
c10f8c9 
)

Remove dead code that was never called anywhere in the codebase:

- Remove unused error constants: ErrNoRetryable, ErrNoVideoFiles
- Remove unused service methods: Database(), GetStats(), GetWorkerCount(),
  UpdateWorkerCount() and ServiceStats struct
- Remove unused queue manager method: GetWorkerCount()
- Remove unused interface methods from QueueManager, QueueOperations,
  and ImportService interfaces
- Remove unused archive function: FormatInt()
- Remove unused file extension functions: AllExtensions(),
  AllExtensionsWith(), AllPossibleExtensions(), WhatIsMostLikelyExtension(),
  sniffTextOrNZB(), isLikelyUTF8()
- Remove unused imports: bufio, os, mimetype, strconv

All removed code was verified as unused through comprehensive grep
searches across the entire codebase.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
drondeseries pushed a commit to drondeseries/altmount that referenced this pull request Jan 2, 2026
@javi11 @claude @drondeseries
refactor(importer): remove unused code from importer package (javi11#186
4dfba7b 
)

Remove dead code that was never called anywhere in the codebase:

- Remove unused error constants: ErrNoRetryable, ErrNoVideoFiles
- Remove unused service methods: Database(), GetStats(), GetWorkerCount(),
  UpdateWorkerCount() and ServiceStats struct
- Remove unused queue manager method: GetWorkerCount()
- Remove unused interface methods from QueueManager, QueueOperations,
  and ImportService interfaces
- Remove unused archive function: FormatInt()
- Remove unused file extension functions: AllExtensions(),
  AllExtensionsWith(), AllPossibleExtensions(), WhatIsMostLikelyExtension(),
  sniffTextOrNZB(), isLikelyUTF8()
- Remove unused imports: bufio, os, mimetype, strconv

All removed code was verified as unused through comprehensive grep
searches across the entire codebase.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
drondeseries pushed a commit to drondeseries/altmount that referenced this pull request Jan 2, 2026
@javi11 @claude @drondeseries
refactor(importer): remove unused code from importer package (javi11#186
42ddbc0 
)

Remove dead code that was never called anywhere in the codebase:

- Remove unused error constants: ErrNoRetryable, ErrNoVideoFiles
- Remove unused service methods: Database(), GetStats(), GetWorkerCount(),
  UpdateWorkerCount() and ServiceStats struct
- Remove unused queue manager method: GetWorkerCount()
- Remove unused interface methods from QueueManager, QueueOperations,
  and ImportService interfaces
- Remove unused archive function: FormatInt()
- Remove unused file extension functions: AllExtensions(),
  AllExtensionsWith(), AllPossibleExtensions(), WhatIsMostLikelyExtension(),
  sniffTextOrNZB(), isLikelyUTF8()
- Remove unused imports: bufio, os, mimetype, strconv

All removed code was verified as unused through comprehensive grep
searches across the entire codebase.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
drondeseries pushed a commit to drondeseries/altmount that referenced this pull request Jan 2, 2026
@javi11 @claude @drondeseries
refactor(importer): remove unused code from importer package (javi11#186
096a302 
)

Remove dead code that was never called anywhere in the codebase:

- Remove unused error constants: ErrNoRetryable, ErrNoVideoFiles
- Remove unused service methods: Database(), GetStats(), GetWorkerCount(),
  UpdateWorkerCount() and ServiceStats struct
- Remove unused queue manager method: GetWorkerCount()
- Remove unused interface methods from QueueManager, QueueOperations,
  and ImportService interfaces
- Remove unused archive function: FormatInt()
- Remove unused file extension functions: AllExtensions(),
  AllExtensionsWith(), AllPossibleExtensions(), WhatIsMostLikelyExtension(),
  sniffTextOrNZB(), isLikelyUTF8()
- Remove unused imports: bufio, os, mimetype, strconv

All removed code was verified as unused through comprehensive grep
searches across the entire codebase.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
@javi11 javi11 deleted the refactor/remove-unused-importer-code branch February 4, 2026 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@javi11