Skip to content

Ibarakov/palette recolor guidance - #13

Merged
onlyexeption merged 17 commits into
mainfrom
ibarakov/palette-recolor-guidance
Oct 26, 2026
Merged

onlyexeption merged 17 commits into
mainfrom
ibarakov/palette-recolor-guidance

Conversation

@onlyexeption

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 Needs a closer look

Cross-theme WPF recoloring recipes require human runtime validation beyond the string-based smoke checks.

No open issues 🎉

What changed in this PR

Updates the C# MCP server’s theme guidance, replacing palette-override skeletons with control-level and extended-theme recoloring instructions.

Changes:

  • Adds theme-specific recoloring recipes and corrects the IG theme class to IgTheme.
  • Aligns tool descriptions, server instructions, and documentation.
  • Adds C#-only smoke checks for generated theme guidance.
File Description
server/​Tools/​ToolDescriptions.cs Updates agent-facing recoloring guidance.
server/​Tools/​ThemeTools.cs Generates theme-specific recipes and corrects IG setup.
server/​ServerInstructions.cs Directs agents to the revised recoloring workflow.
server/​README.md Updates packaged-server documentation.
scripts/​smoke-test.ts Checks IG setup and recoloring output.
README.md Updates the public feature summary.

Review effort: Balanced

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 Needs a closer look

The expanded WPF recoloring recipes need human runtime validation across supported themes beyond the string-based smoke checks.

No open issues 🎉

Review effort: Balanced

Marina-L-Stoyanova and others added 7 commits October 20, 2026 12:03
…me-resource-paging

feat(theme): page large theme files and find entries instead of cutting
Updated regex to correctly match escaped Unicode sequences in JSON strings.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
…ns-per-namespace

Ibarakov/xmlns per namespace
onlyexeption and others added 2 commits October 22, 2026 10:37
…ocs-ci-hardening

feat: update MCP publisher version and checksum in workflows; enhance…

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The resource filter needs runtime length validation, and installed-package smoke tests must read data from the target installation rather than the checkout.

2 open issues

Review effort: Balanced

Comment thread server/Tools/ThemeTools.cs Resolved
Comment thread scripts/smoke-test.ts Resolved
…server's startup data directory'

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@onlyexeption
onlyexeption dismissed Marina-L-Stoyanova’s stale review October 26, 2026 12:44

The merge-base changed after approval.

@onlyexeption
onlyexeption dismissed Marina-L-Stoyanova’s stale review October 26, 2026 12:52

The merge-base changed after approval.

@onlyexeption
onlyexeption merged commit e1d7007 into main Oct 26, 2026
7 checks passed
@onlyexeption
onlyexeption deleted the ibarakov/palette-recolor-guidance branch October 26, 2026 12:54
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.

3 participants