Skip to content

Add per-category and per-mod output grouping - #4

Closed
elseform wants to merge 1 commit into
noisethanks:masterfrom
elseform:feature-per-category
Closed

elseform wants to merge 1 commit into
noisethanks:masterfrom
elseform:feature-per-category

Conversation

@elseform

@elseform elseform commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Mod Output Mode can now split compressed output across multiple mod
folders instead of one flat folder:

  • Per texture category: ATAK - Normal Maps, ATAK - Diffuse, etc
  • Per source mod: ATAK - SomeMod
  • Both combined: ATAK - SomeMod - Normal Maps

Why split

  • Per-category makes A/B testing a single category easier
  • Per-mod makes updating easier once mod-update drops

Both are opt-in toggles in Settings. Per-category defaults on
(matches prior behavior on this branch); per-mod defaults off.

Scan exclusion glob (ATAK - *) now matches all three folder-name
shapes so grouped output is never rescanned as a source mod.

Testing

  • go test ./... — added internal/tui/screens/percategory_test.go
    covering all four grouping combinations (off / category / mod / both),
    the scan exclusion glob, and incremental-skip isolation between
    category output folders
  • Manually verified in the TUI: ran a category through Mod Output Mode,
    confirmed folder created, confirmed rerun skips already-compressed files

Known follow-up

Settings screen has no scroll support (pre-existing, not introduced here)
— these two new toggles push it further past typical terminal height on
short terminals. #5

Mod Output Mode can now split compressed output across multiple mod
folders instead of one flat folder:

- Per texture category: ATAK - Normal Maps, ATAK - Diffuse, etc
- Per source mod: ATAK - SomeMod
- Both combined: ATAK - SomeMod - Normal Maps

Why split at all:
- Per-category makes A/B testing a single category trivial - drop or
  reorder just "ATAK - Normal Maps" in MO2 without touching the rest
- Per-mod makes updating a source mod's textures trivial - delete
  just that mod's output folder and rerun

Both are opt-in toggles in Settings. Per-category defaults on
(matches prior behavior on this branch); per-mod defaults off.

Scan exclusion glob (ATAK - *) now matches all three folder-name
shapes so grouped output is never rescanned as a source mod.
@elseform

Copy link
Copy Markdown
Contributor Author

B6CDAA30-B6A6-441B-918C-4242B045EDC57B048729-0D46-42D1-A1CF-AE215727DC79

@elseform
elseform marked this pull request as ready for review August 12, 2026 12:03
@elseform elseform closed this Aug 12, 2026
@elseform
elseform deleted the feature-per-category branch August 12, 2026 14:41
@elseform

Copy link
Copy Markdown
Contributor Author

This PR was closed by mistake — GitHub auto-closed it as a side effect of renaming the source branch (feature-per-categoryfeat/per-category), which doesn't carry over cross-fork PR associations. The correct, up-to-date PR is #6.

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