Skip to content

docs: propose the Policy Store requirements resolver - #1483

Open
Chaz Gordish (ChazGo) wants to merge 1 commit into
microsoft:mainfrom
ChazGo:chazgo-mxc-policy-resolver-spec
Open

Chaz Gordish (ChazGo) wants to merge 1 commit into
microsoft:mainfrom
ChazGo:chazgo-mxc-policy-resolver-spec

Conversation

@ChazGo

@ChazGo Chaz Gordish (ChazGo) commented Oct 13, 2026 •

Copy link
Copy Markdown

📖 Description

Propose one resolveToolRequirements operation that returns suggested container requirements and the original input indexes not completely resolved.

The API spec covers inputs, outputs, matching, errors, TypeScript/Rust/.NET signatures, and one Zava Agent example. The developer spec defines catalog authoring, dependency composition, validation, packaging, and the implementation test plan.

This is a documentation-only proposal for API review before implementation. It includes no SDK implementation or API stubs. It is not part of MXC 1.0; no later release is committed.

🔗 References

Supersedes #1309 for a fresh review of the resolver-only proposal. The earlier discussion remains there for reference.

🔍 Validation

  • Strict TypeScript compilation of the declarations and example against the current SDK request types.
  • Declaration parity with the local resolver-only review stubs, including Rust/.NET signature excerpts.
  • Example caller-flow checks using fake SDK/application hooks, covering saved settings, restrictions, absent and empty results, error propagation, and explicit working directory.
  • JSON example preservation, 57 relative links/anchors, and git diff --check.

These are document, type, and example checks. The resolver is not implemented; native resolution and containment were not tested.

✅ Checklist

No issue is linked. Implementation instructions and Cargo.lock are unchanged.

📋 Issue Type

  • Bug fix
  • Feature
  • Task

GitHub Actions runs the PR validation build automatically. The ADO pipeline
(MXC-PR-Build) is the Azure version of the PR pipeline, kept in parity with the GitHub
Actions build; it runs on merge to main, and Microsoft reviewers with write access can trigger it
on a PR with /azp run. See pull request builds.

If the dependency-feed-check check fails on a new dependency, the crate must be added to
the feed before the PR can pass. See pull request builds
for the steps.

Microsoft Reviewers: Open in CodeFlow

Define the resolver-only API, complete-input accounting, matching rules, one Zava example, and the catalog implementation design.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings October 13, 2026 19:16
@ChazGo
Chaz Gordish (ChazGo) requested a review from a team as a code owner October 13, 2026 19:16

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

Security-sensitive policy composition and unresolved cross-SDK and platform-context semantics require designated human API review.

3 open findings
What changed in this PR

Proposes a documentation-only Policy Store resolver API and its catalog, validation, packaging, and testing design.

Changes:

  • Defines resolveToolRequirements behavior and cross-SDK signatures.
  • Specifies catalog matching, composition, governance, and validation.
  • Adds proposal links to the root README.
File Description
README.md Links the proposal documents.
docs/​mxc-policy-store.md Defines catalog and implementation requirements.
docs/​mxc-policy-store-api.md Defines the proposed public API contract and example.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment on lines +135 to +137
These proposed declarations are exported from `@microsoft/mxc-sdk/v1`.
Rust/.NET use equivalent versioned types and reuse their existing SDK policy
sections rather than defining a parallel policy model.
export interface ResolveContext {
projectRoot?: string;
symbols?: Record<string, string>;
platform?: CatalogPlatform;
Comment thread docs/mxc-policy-store.md
For caller-facing review, start with the [API spec](mxc-policy-store-api.md).

This document does not restate general MXC sandboxing concepts already covered
by the [v1 SDK reference](https://github.com/microsoft/mxc/blob/894f4c159705f5f470727e4fa1e363a2abec88f1/docs/reference/node/v1/README.md) or

This branch has not been deployed

No deployments
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