Repository navigation
Assistant v4 - #1926
Merged
Merged
Assistant v4#1926
Conversation
- move Key-tier services into services/assistant/provider/ - answerers are plain functions: askServer, askProvider - key types, linkable commands and the operation index are built at runtime, not parsed into context.generated.ts - drop the redundant map-id tool guard and the byName link fallback - shared errorText util; drop tier.ts, origins.ts, entity-types.ts - PRD footer and architecture doc match the code Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Azgaar
added a commit
that referenced
this pull request
Oct 5, 2026
* [Draft] Style editor migration (#1884) * docs * init * style editor UI * ui cleanup * update docs * cleanup * refactor * organize * simplify model * cleanup * sort * scheme refinement * cleanup * cleanup * cleanup * fixes * fixes * refactor * reorganize data + migration * cleanup, reogragization * cleanup * fixes * change zoomFontSize * refactor + fixes * preview improvement * fixes * lint fix * fix * Icons rework (#1903) * prd v1 * prd v2 * implementation v1 * implemnetation v2 * implementation v3 * implementation 4 * fixes * More icons (#1906) * more illustrated icons * icons cleanup * zoom per layer, not on viewbox * style cleanup * atlas preset * new stuff * fix * Choose a map save location once, then save directly to it (#1921) * fix: restore citadel feature key in burg and burg group editors (#1912) The citadel icon rename to "palace" also caught the burg editor's data-feature attribute and the burg group feature list, so toggling the citadel wrote burg.palace and left burg.citadel unchanged. Fixes #1902 * fix: escape the market name in the Markets Overview remove prompt (#1915) Fixes #1913 * Offer to remove the market when unsetting a market center's plaza (#1914) * fix: offer to remove the market when unsetting a market center's plaza Turning the plaza off in the burg editor left the burg as its market's center, so the burg still could not be deleted. Ask to remove the market instead; cancelling keeps the plaza. Fixes #1899 * docs: explain how to delete a market center burg * fix: escape the market name in the remove-market prompt * fix: correct typos in tooltip text (#1918) - Fix 'descrese' → 'decrease' in urbanization rate tooltip (units-editor.ts) - Fix 'back map the generator' → 'back to the map generator' in emblem editor tooltip (emblems-editor.ts) Closes #1909, #1917 * feat: choose a save location for map files Port the save-location picker proposed in #1514 to the current services API. Open the picker before serialization and load the save service eagerly so the first save retains browser user activation. Handle Ctrl/Cmd+S on keydown without triggering a layer shortcut on release. Keep download fallback and desktop feedback, escape chosen filenames, and report permission/write failures through the save dialog. Cover cancellation, stream failures, feedback, and hotkey behavior with regression tests. Co-authored-by: Spencer Pruitt <spencerpruitt94@gmail.com> * feat: reuse the chosen map file for quick saves * test: cover picker saves and explicit download fallback --------- Co-authored-by: RoomWithOutRoof <166608075+Jah-yee@users.noreply.github.com> Co-authored-by: Spencer Pruitt <spencerpruitt94@gmail.com> * Icons library (#1920) * goods - custom icons fix * icon library start * icons selector - next * improve UI * fixes * relief icons * relief rules * fixes * fixes * docs * Emblems in the icon library, Armoria round-trip, icon simplifications - Charges become per-category icon sets; picture emblems and library charges - Armoria edit sessions, drawability check, shields/compony/diapers ported - Custom icons rebuild their own symbol on change; Icons.require for exports - IconSets.fileOf index; sets declare their picker group - parseArmoria throws author-facing errors; shared inescutcheonShape Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * simplification --------- Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com> * update docs * Diplomacy painter add-on (#1907) * Update diplomacy-editor.ts * Update diplomacy-editor.ts * Update diplomacy-editor.ts * Update diplomacy-editor.ts Updated true paint function with default Change Relations menu * omnibar update * diplomacy editor * AI assistant v3 (#1808) Add the This map assistant with provider selection, map questions, note editing, undo, conversation history, and supporting tests and documentation. * Assistant v4 (#1926) * rename * PRD * widgets, compact context * fixes * more operations * operations * fixes * don't destroy on close * keep scroll * simplifications * fixes * simplify * fixes * fixes * simplify assistant: provider folder, runtime-derived context - move Key-tier services into services/assistant/provider/ - answerers are plain functions: askServer, askProvider - key types, linkable commands and the operation index are built at runtime, not parsed into context.generated.ts - drop the redundant map-id tool guard and the byName link fallback - shared errorText util; drop tier.ts, origins.ts, entity-types.ts - PRD footer and architecture doc match the code Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * simplification * fixes * fixes --------- Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com> * fixed * fixes * fix(nix): refresh npm dependency hash Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * fixes * fixes * changes * assistant - style update * icons fix * relief fixes * assistant to see images * bug fixes * fixes * fixes * fix - relief to have stable stroke width * fixes * fixes, update notes * fix(nix): include docs bundled by the assistant in the source fileset Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * test(e2e): drop relief stroke-width from expectations The relief layer no longer sets stroke-width (023ec6f); icons carry their own. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> --------- Co-authored-by: barrulus <b@rry.im> Co-authored-by: RoomWithOutRoof <166608075+Jah-yee@users.noreply.github.com> Co-authored-by: Spencer Pruitt <spencerpruitt94@gmail.com> Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com> Co-authored-by: WebHead89 <davis.webster@hotmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description