summaryrefslogtreecommitdiff
path: root/internal/service/write.go
diff options
context:
space:
mode:
authorPaul Buetow <paul@buetow.org>2026-05-04 08:26:08 +0300
committerPaul Buetow <paul@buetow.org>2026-05-04 08:26:08 +0300
commit81737024f1d39c022d7901de380a5d87df70d4b4 (patch)
tree3c7a6b79cdb951a30a00e9da9e783a7bd3e409cc /internal/service/write.go
parentb15b271dd350e97396f0a7096c71dabc045c5d83 (diff)
task 9: Centralize file extension and media type mappings in internal/mediatype
Create a new internal/mediatype package with a single source of truth for: - Extension-to-media-type mappings (TypeForExt) - Extension-to-MIME mappings (MIMETypeForExt) - Supported extension checks (IsSupportedExt, IsImageExt, IsCoverImageExt) Replace duplicated logic across: - internal/service/service.go (supportedExtensions, guessMediaType) - internal/scanner/scanner.go (mediaExtensions, imageExtensions, mediaTypeFromExt) - internal/api/handlers.go (probe.MimeTypeForFilename) - internal/probe/probe.go (imageExtensions, isImagePath) - internal/probe/remux.go (MimeTypeForFilename) Divergent defaults unified: both scanner and service now default unknown extensions to video (model.MediaTypeVideo) via mediatype.TypeForExt. Update tests to use mediatype package and remove obsolete scanner tests. Update AGENTS.md to reflect the new package.
Diffstat (limited to 'internal/service/write.go')
-rw-r--r--internal/service/write.go5
1 files changed, 3 insertions, 2 deletions
diff --git a/internal/service/write.go b/internal/service/write.go
index af0c839..190add8 100644
--- a/internal/service/write.go
+++ b/internal/service/write.go
@@ -10,6 +10,7 @@ import (
"strings"
"codeberg.org/snonux/player/internal/clock"
+ "codeberg.org/snonux/player/internal/mediatype"
"codeberg.org/snonux/player/internal/model"
"codeberg.org/snonux/player/internal/probe"
"codeberg.org/snonux/player/internal/repository"
@@ -67,7 +68,7 @@ func (s *writeService) UploadMedia(ctx context.Context, setID, userID int64, fil
return nil, err
}
- if !isSupportedExtension(filename) {
+ if !mediatype.IsSupportedExt(filename) {
return nil, fmt.Errorf("%w: %s", ErrUnsupportedExtension, filepath.Ext(filename))
}
@@ -163,7 +164,7 @@ func (s *writeService) saveUploadedMedia(ctx context.Context, setID int64, path
RelPath: filepath.Base(path),
FileName: filepath.Base(path),
AbsPath: path,
- Type: guessMediaType(filepath.Base(path)),
+ Type: mediatype.TypeForExt(filepath.Base(path)),
FileSizeBytes: n,
CreatedAt: now,
}