Skip to content

A package can say AI helped make it, and which tools - #74

Merged
McCal-Codes merged 1 commit into
mainfrom
manifest-ai-assisted
Sep 22, 2026
Merged

McCal-Codes merged 1 commit into
mainfrom
manifest-ai-assisted

Conversation

@McCal-Codes

Copy link
Copy Markdown
Owner

What this changes

Adds an optional aiAssisted field to manifest schema v1, so a Market package can declare that AI helped make it:

"aiAssisted": { "tools": ["Claude"], "note": "Drafted the colours; I picked the final ones." }
  • tools: 1 to 5 names, 60 characters each, no blanks or repeats. note: optional text, translatable.
  • The Market parser reads it with the same limits as the schema, so SchemaConformanceTest's differential fuzz still agrees.
  • Folio 0.6.6 skips fields it doesn't know rather than refusing the package, so packages using it still install there.
  • The format table, rule AI-6 and the SDK rules describe it. Until the Market shows the field, AI-6 still asks for "AI-assisted (tool name)." at the start of description.

Not in this PR: showing aiAssisted on the package page and at install (the remaining AI gap).

How I checked it

  • ./gradlew :app:testDebugUnitTest :market:testDebugUnitTest :app:lintDebug :market:lintDebug :app:assembleDebug passes.
  • New PackageParserTest case: reads the field; refuses a missing, empty, blank, duplicate, over-long or sixth tool, and a non-object.
  • tools/folio-pkg.py validate accepts a package with the field and refuses an empty tools; docs/sdk/source still validates.

Companion template PR updates its vendored schema and instructions.

Adds aiAssisted to manifest schema v1: { tools: 1-5 names, note?: text }.
The parser reads it with the same limits, so the conformance fuzz agrees.
Folio 0.6.6 skips the field rather than refusing the package. Until the
Market shows it, AI-6 also keeps the description prefix.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 44b9ee5fde

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

val depiction: String? = null,
val license: String? = null,
val description: LocalizedText? = null,
val aiAssisted: AiAssisted? = null,

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Do not land an AI-authored new feature

The commit metadata identifies Codex as the author, and this added model field is part of a new manifest-format feature rather than a bug fix. The repository's pre-existing AI-3 rule (docs/standards/ai-contributions.md:24-26) explicitly forbids using an AI agent for new features and directs agents to open an issue instead, so this change cannot be accepted under the project's contribution policy.

Useful? React with 👍 / 👎.

@McCal-Codes
McCal-Codes merged commit 096528a into main Sep 22, 2026
2 checks passed
@McCal-Codes
McCal-Codes deleted the manifest-ai-assisted branch September 22, 2026 15:12
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.

1 participant