Repository navigation
docs: name Micrate, Gemma, and Mica in the ecosystem agent context - #1417
Draft
crimson-knight wants to merge 2 commits into
Draft
crimson-knight wants to merge 2 commits into
crimson-knight wants to merge 2 commits into
Conversation
Amber V2 parses request bodies as JSON and form data (plus the bounded CBOR and COSE schema formats). The XML parser added in the schema params work required Crystal's `xml` module, which made every V2 application link libxml2 and let any client send a document for libxml2 to parse. - Remove Amber::Schema::Parser::XMLParser, its example, README, and specs. - Answer application/xml, text/xml, and any +xml request body with 415 Unsupported Media Type, including when a schema declares the type, and stop sniffing `<`-prefixed bodies as XML in standalone parsing. - Add ParserRegistry.register_body_parser so an application can opt in to a body format such as XML with one call at boot. XML support returns later as an opt-in parser built on a pure-Crystal XML library. - Document the breaking change in CHANGELOG, the migration guide, and the schema guide.
CLAUDE.md ships to applications as the amber--docs agent skill. Its external-dependency list now covers Micrate and Gemma and introduces Mica, the pure-Crystal parsing library in development that is the planned XML parser for ParserRegistry.register_body_parser. The schema guide names Mica where it describes XML returning as an opt-in parser.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Stacked on #1414. This branch is built on
remove/xml-request-parser, so until #1414 merges, the diff also shows its commit (217bcf2). Review only the top commit, 17f9d68. Docs only; there are no code changes.Why
V2 apps receive
CLAUDE.mdas theamber--docsagent skill, distributed by shards-alpha throughai_docs. Its "External Dependencies" list named Grant, the CLI and Asset Pipeline, but not Micrate, Gemma or the coming parsing library. An agent working in a V2 app therefore had no ecosystem map for migrations, attachments or XML.What changes
CLAUDE.md"External Dependencies" now covers three more shards, plus one link:amber database migrate,rollbackandstatus.ParserRegistry.register_body_parser. Other formats are planned, not built, and Amber does not depend on it.docs/guides/schema-api.md: the sentence on XML returning as an opt-in parser now names Mica and says it has no release yet.Verification
crystal-alpha build --no-codegen src/amber.cron this branch reports 0 errors and 0 warnings. No Crystal source changed.