Skip to content

feat: copilot variant styling improvements#5682

Merged
compulim merged 7 commits intomainfrom
feat/qol-improvements
Feb 3, 2026
Merged

feat: copilot variant styling improvements#5682
compulim merged 7 commits intomainfrom
feat/qol-improvements

Conversation

@OEvgeny
Copy link
Collaborator

@OEvgeny OEvgeny commented Jan 31, 2026

Fixes #

Changelog Entry

  • Improved adaptive cards rendering in copilot variant, in PR #5682, by @OEvgeny

Description

This includes some improvements for the copilot variant rendering adaptive cards.

This also introduces a new build pipeline for test assets so we don't depend much on esm.sh stability.

Specific Changes

  • I have added tests and executed them locally
  • I have updated CHANGELOG.md
  • I have updated documentation

Review Checklist

This section is for contributors to review your work.

  • Accessibility reviewed (tab order, content readability, alt text, color contrast)
  • Browser and platform compatibilities reviewed
  • CSS styles reviewed (minimal rules, no z-index)
  • Documents reviewed (docs, samples, live demo)
  • Internationalization reviewed (strings, unit formatting)
  • package.json and package-lock.json reviewed
  • Security reviewed (no data URIs, check for nonce leak)
  • Tests reviewed (coverage, legitimacy)

@OEvgeny OEvgeny force-pushed the feat/qol-improvements branch from a1f9a08 to 389edd4 Compare January 31, 2026 00:56
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR improves the Fluent Theme “copilot” variant styling (notably around adaptive cards) and adds a new “test-assets” build pipeline so HTML tests can rely less on esm.sh for runtime dependencies.

Changes:

  • Updated Fluent Theme CSS for copilot variant (image sizing, adaptive card typography/button styling, background clipping).
  • Added a new packages/test/test-assets workspace that builds browser-ready ESM bundles for selected dependencies via esbuild.
  • Updated test harness HTML import maps and Docker/static serving config to consume the new locally-built test assets.

Reviewed changes

Copilot reviewed 50 out of 70 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
serve-test.json Adds route mappings to serve the built test-assets/out files via the test web server.
packages/test/test-assets/package.json Introduces the new internal “test-assets” workspace and build scripts.
packages/test/test-assets/module/react.js Provides an alias module wrapper for React in the bundling flow.
packages/test/test-assets/module/react-scheduler.js Adds a scheduler alias wrapper used by the bundling flow.
packages/test/test-assets/module/react-jsx-runtime.js Adds a JSX runtime alias wrapper used by the bundling flow.
packages/test/test-assets/module/react-dom/client.js Adds a ReactDOM client alias wrapper used by the bundling flow.
packages/test/test-assets/module/react-dom.js Adds a ReactDOM alias wrapper used by the bundling flow.
packages/test/test-assets/entry/react.js Defines a React entry module for bundling/export shaping.
packages/test/test-assets/entry/react-jsx-runtime.js Defines a JSX runtime entry module for bundling.
packages/test/test-assets/entry/react-dom/client.js Defines a ReactDOM client entry module for bundling.
packages/test/test-assets/entry/react-dom.js Defines a ReactDOM entry module for bundling/export shaping.
packages/test/test-assets/entry/@fluentui/tokens.js Defines a Fluent UI tokens entry module for bundling.
packages/test/test-assets/entry/@fluentui/react-provider.js Defines a Fluent UI provider entry module for bundling.
packages/test/test-assets/Makefile Adds the esbuild-based build pipeline to emit browser ESM bundles into out/.
packages/test/test-assets/.gitignore Ignores out/, node_modules/, and the downloaded make.com.
packages/fluent-theme/src/components/theme/Theme.module.css Adds a copilot variant CSS custom property affecting image bubble sizing.
packages/fluent-theme/src/components/activity/PartGroupingDecorator.module.css Adjusts attachment spacing and improves adaptive card styling in copilot variant; adds background clipping.
package.json Adds packages/test/test-assets to the root workspace list.
package-lock.json Locks dependencies for the new workspace and updates the monorepo lock accordingly.
docker-compose-wsl2.yml Mounts packages/test/test-assets/out into the web container for HTML tests.
tests/html2/side-by-side/index.html Switches Fluent imports to locally built assets; adjusts import map entries.
tests/html2/part-grouping/status.html Switches Fluent imports to locally built assets; adjusts imports.
tests/html2/part-grouping/position.html Switches Fluent imports to locally built assets; adjusts imports.
tests/html2/part-grouping/navigation.html Switches Fluent imports to locally built assets; adjusts imports.
tests/html2/part-grouping/keyboard.html Adjusts import map to use locally built Fluent assets and JSX runtime mapping.
tests/html2/part-grouping/index.html Switches Fluent imports to locally built assets; adjusts imports.
tests/html2/markdown/math/layout.scroll.html Switches Fluent imports to locally built assets; adjusts imports.
tests/html2/markdown/codeBlockCopyButton/adaptiveCards/layout.copilot.html.snap-1.png Updates snapshot image to reflect new copilot/adaptive card rendering.
tests/html2/linkDefinition/badge.html Switches Fluent imports to locally built assets; adjusts imports.
tests/html2/fluentTheme/uiState.html Switches Fluent imports to locally built assets; adjusts imports.
tests/html2/fluentTheme/uiState.blueprint.nonEnglish.html Switches Fluent imports to locally built assets; adjusts imports.
tests/html2/fluentTheme/fallback.html Switches Fluent imports to locally built assets; adjusts imports.
tests/html2/fluentTheme/defaultFeedback.activity.html Switches Fluent imports to locally built assets; adjusts imports.
tests/html2/feedbackForm/feedback.form.scroll.html Removes unused Fluent import map entry (keeps React-only imports).
tests/html2/copyButton/layout.html Switches Fluent imports to locally built assets; adjusts imports.
tests/html2/citation/url.html Switches Fluent imports to locally built assets; adjusts imports and JSX runtime mapping.
tests/html2/basic/customIcons.html Removes unused Fluent import map entry (keeps React-only imports).
tests/html2/attachment/adaptiveCard/custom.skip.html Adds a new test page validating adaptive card styling/behavior with the copilot variant.
tests/html2/attachment/adaptiveCard/custom.fluent.html Adds a redirect page for fluent variant coverage of the new test.
tests/html2/attachment/adaptiveCard/custom.fluent.dark.html Adds a redirect page for fluent+dark coverage of the new test.
tests/html2/attachment/adaptiveCard/custom.copilot.html Adds a redirect page for copilot variant coverage of the new test.
tests/html2/attachment/adaptiveCard/custom.copilot.dark.html Adds a redirect page for copilot+dark coverage of the new test.
tests/html2/activity/message-status.html Switches Fluent imports to locally built assets; adjusts JSX runtime mapping.
tests/html2/activity/collapsible.html Switches Fluent imports to locally built assets; adjusts imports.
tests/assets/esm/test.js Adds ESM test bootstrapping helpers for HTML2 tests.
tests/assets/esm/postActivity.js Adds an ESM helper to post activities to the adapter in tests.
tests/assets/custom-element/test-shell.ce.html Adds a new test shell custom element used by the new adaptive card test.
tests/assets/custom-element/keyboard-event.ce.html Updates custom element registration logic to use customElements.getName.
tests/assets/custom-element/custom-element.js Removes the previous getCustomElementName workaround and related bookkeeping.
CHANGELOG.md Adds a changelog entry for improved copilot adaptive card rendering.
.github/workflows/pull-request-validation.yml Ensures the built test-assets/out directory is included in the Docker artifact.
.dockerignore Includes packages/test/test-assets/out in Docker build context for tests.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@OEvgeny OEvgeny requested a review from compulim February 2, 2026 19:38
Copilot AI and others added 2 commits February 3, 2026 12:13
* Initial plan

* Revert make.com vendoring, add SHA verification and download from Internet

Co-authored-by: compulim <1622400+compulim@users.noreply.github.com>

* Security: Move chmod after SHA verification

Co-authored-by: compulim <1622400+compulim@users.noreply.github.com>

* Remove makeComSha256 field, hardcode SHA in build:pre:setup script

Co-authored-by: compulim <1622400+compulim@users.noreply.github.com>

* Reorder build:pre script: local-dependencies before setup

Co-authored-by: compulim <1622400+compulim@users.noreply.github.com>

* Use more specific .gitignore patterns with leading/trailing slashes

Co-authored-by: compulim <1622400+compulim@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: compulim <1622400+compulim@users.noreply.github.com>
@compulim compulim merged commit e78ed2d into main Feb 3, 2026
30 checks passed
@compulim compulim deleted the feat/qol-improvements branch February 3, 2026 21:20
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