Skip to content

Add license string support in aw.yml manifest parsing and schema#36583

Draft
Copilot wants to merge 2 commits into
mainfrom
copilot/add-license-field-support
Draft

Add license string support in aw.yml manifest parsing and schema#36583
Copilot wants to merge 2 commits into
mainfrom
copilot/add-license-field-support

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jun 3, 2026

aw.yml package manifests did not accept a license field, so manifests including license metadata failed strict schema validation. This change adds first-class support for license as a string and propagates it through manifest parsing.

  • Schema update

    • Added license to pkg/parser/schemas/aw_manifest_schema.json as type: string, preserving root-level strictness (additionalProperties: false) while allowing the new field.
  • Manifest model + parser

    • Added License to internal manifest and resolved package structs in pkg/cli/add_package_manifest.go.
    • Parsed license from the manifest root when present and surfaced it in resolved package metadata.
  • Coverage for expected behavior

    • Extended pkg/cli/add_package_manifest_test.go to verify:
      • valid string license is accepted and mapped (license: MIT)
      • non-string license values are rejected by schema validation
name: Repo Assist
license: MIT
includes:
  - workflows/review.md

Copilot AI and others added 2 commits June 3, 2026 06:29
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title Add aw.yml license field support Add license string support in aw.yml manifest parsing and schema Jun 3, 2026
Copilot AI requested a review from pelikhan June 3, 2026 06:31
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