Skip to content

fix(ingestion): nested folder Move paths for K2 prestige advisories#195

Open
oldrepublicwizard wants to merge 2 commits into
feat/k2-merged-download-install-smokefrom
feat/k2-prestige-nlp-folder-move
Open

fix(ingestion): nested folder Move paths for K2 prestige advisories#195
oldrepublicwizard wants to merge 2 commits into
feat/k2-merged-download-install-smokefrom
feat/k2-prestige-nlp-folder-move

Conversation

@oldrepublicwizard

Copy link
Copy Markdown
Owner

Summary

  • Add BuildFolderMoveSources / ExpandFolderMoveSources for NLP folder advisories (nested <<modDirectory>>/*…/folder/* paths plus slug/fuzzy variants)
  • Wire folder expansion into NaturalLanguageInstructionParser, K2FullGuideRoundTripHelper, and Prestige/Silent Sion regression tests
  • Document folder Move behavior in cli-selection-semantics.md

Stacks on #191 (feat/k2-merged-download-install-smoke).

Test plan

  • DraftInstructions_PrestigeClassAdvisory_IncludesNestedFolderSearch
  • K2FullGuideFixture_IngestedPrestige_InstallsFromNestedExtractFolder
  • K2FullGuideFixture_IngestedMultiModInstallSmoke_InstallsViaModSelect
  • DraftInstructions_SilentSionMoveJust_IncludesNestedSearchPaths

Expand NLP folder Move drafts with nested wildcards and slug/fuzzy folder variants so Prestige Class installs resolve jedimaster_sithlord fixes after real zip extract nesting.
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

Extend the #191 round-trip harness with Prestige Class folder-Move coverage, overlay helper assertions, and agent script --mod routing for network smoke.
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