Skip to content

Add Essentials.AI to main README and create product README#211

Merged
jfversluis merged 6 commits into
mainfrom
docs/add-essentials-ai-to-readme
May 2, 2026
Merged

Add Essentials.AI to main README and create product README#211
jfversluis merged 6 commits into
mainfrom
docs/add-essentials-ai-to-readme

Conversation

@jfversluis
Copy link
Copy Markdown
Member

Following the merge of #171 (Microsoft.Maui.Essentials.AI), this adds the new product to the repo documentation.

Changes

  1. Main README. Added Essentials.AI section under Products, between DevFlow and Getting Startedmd
  2. src/AI/README. New product README with features, quick start, package table, build instructions, and architecture overviewmd

Every other product (Cli, Comet, Go, DevFlow) has a section in the main README and a product-level README. This brings Essentials.AI in line.

Add the new Microsoft.Maui.Essentials.AI product to the main README's
Products section, and create src/AI/README.md with package description,
quick start, build instructions, and architecture overview.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 2, 2026 11:40
Add to both AGENTS.md and copilot-instructions.md that new products
must have:
- A product-level README.md packed into the NuGet (with
  PackRepoRootReadme=false to avoid duplicates)
- An entry in the repo-root README.md under Products

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
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 adds repository-level documentation for the new Microsoft.Maui.Essentials.AI product introduced in #171, so the AI package is represented alongside the other MAUI Labs products and has its own product README.

Changes:

  • Adds an Essentials.AI product section to the root README.md.
  • Introduces src/AI/README.md with features, quick start, build notes, and architecture details.
  • Aligns Essentials.AI with the repo’s existing product-documentation structure.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
README.md Adds a high-level Essentials.AI product summary to the repo’s main product catalog.
src/AI/README.md Adds the product-specific README covering setup, build guidance, and architecture.

Comment thread src/AI/README.md Outdated
Comment thread src/AI/README.md Outdated
Comment thread src/AI/README.md Outdated
Comment thread README.md Outdated
jfversluis and others added 3 commits May 2, 2026 13:44
Align with NuGet README: add platform support table, requirements
section, and experimental status warning.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
 AddSingleton<IChatClient>
  (package only brings Abstractions, not full Extensions.AI helpers)
- Clarify Windows build requires CI=true for prebuilt artifact path
- Separate NL embeddings from Apple Intelligence (uses NaturalLanguage
  framework, not Foundation Models)
- Reorder Mac Catalyst in requirements for clarity

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Add content checklist to both AGENTS.md and copilot-instructions.md:
name, features, platform matrix, quick start, packages, build
instructions, architecture, requirements, and status warning.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 2, 2026

Expert Code Review — PR #211

Methodology: 3 independent reviewers with adversarial consensus

4 findings posted as inline comments (3 🟡 moderate, 1 🟢 minor)

All findings are posted inline on the relevant lines — no overflow table needed.

Discarded findings (single reviewer, no consensus)

Finding File Reviewer Reason
Cross-platform targets may mislead Android/Windows users src/AI/README.md L11 Reviewer 1 only Failed follow-up — both other reviewers confirmed existing platform table already addresses this
Relative link ../../docs/ai/... broken inside NuGet package src/AI/README.md L53 Reviewer 1 only Below 3-finding cap; no follow-up
tvOS omitted from feature descriptions despite [SupportedOSPlatform] README.md / src/AI/README.md Reviewer 2 only Below 3-finding cap; no follow-up
Mac Catalyst 26+ requirement not mentioned alongside iOS/macOS 26+ src/AI/README.md L11 Reviewer 3 only Below 3-finding cap; no follow-up
NL embeddings attributed to Apple Intelligence rather than Natural Language framework README.md L76 Reviewer 3 only Below 3-finding cap; no follow-up

CI Status

  • license/cla — passed
  • 🔄 build-macos / build (macos-latest) — in progress at time of review

Test Coverage

Documentation-only PR — no code changes requiring test coverage.

Generated by Expert Code Review · 3 independent reviewers with adversarial consensus

Warning

⚠️ Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • learn.microsoft.com

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "learn.microsoft.com"

See Network Configuration for more information.

Generated by Expert Code Review (auto) for issue #211 · ● 6.3M ·

Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Expert Code Review: 4 findings posted inline (3 moderate, 1 minor). See the summary comment for full details.

Warning

⚠️ Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • learn.microsoft.com

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "learn.microsoft.com"

See Network Configuration for more information.

Generated by Expert Code Review (auto) for issue #211 · ● 6.3M

Comment thread src/AI/README.md
Comment thread src/AI/README.md
Comment thread src/AI/README.md
Comment thread .github/copilot-instructions.md Outdated
- Add note to product README linking to NuGet consumer README
- Fix AGENTS.md and copilot-instructions.md: clarify that NuGet README
  lives next to the csproj (not product root), and explain the
  two-README pattern (contributor vs consumer)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@jfversluis jfversluis merged commit f9649d9 into main May 2, 2026
2 checks passed
@jfversluis jfversluis deleted the docs/add-essentials-ai-to-readme branch May 2, 2026 11:55
@jfversluis jfversluis added this to the v0.1.0-preview.8 milestone May 6, 2026
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.

2 participants