feat: classify and gate publishable templates - #59
Closed
xiongxz wants to merge 6 commits into
Closed
Conversation
Co-authored-by: bruinxz <bruinxz@bruinxzdeMacBook-Pro.local>
Co-authored-by: bruinxz <bruinxz@bruinxzdeMacBook-Pro.local>
Co-authored-by: bruinxz <bruinxz@bruinxzdeMacBook-Pro.local>
Co-authored-by: bruinxz <bruinxz@bruinxzdeMacBook-Pro.local>
* feat: add Insight Flow Agent chat template * refactor: move agent config behind chat UI * refactor: simplify agent key storage * fix: harden streaming agent template * fix: normalize agent endpoint hosts * fix: surface function errors in the template --------- Co-authored-by: bruinxz <bruinxz@bruinxzdeMacBook-Pro.local>
|
Thanks for the PR, @xiongxz! A quick note on our workflow: we ask contributors to open an issue first, get it assigned, then submit a PR that links it (e.g. "Closes #123"). This PR isn't linked to any issue. It'll still be reviewed, but please open an issue and claim it (comment that you'd like it assigned to you) so the work is tracked. |
Author
|
Companion PRs for the same delivery line:
|
Author
|
Closing as superseded: this PR was mistakenly opened against upstream and therefore included five pre-existing fork commits outside this delivery. The corrected one-commit, task-scoped PR targets the production fork baseline at lexmount#6. No changes from this oversized PR were merged. |
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.
Summary
Compatibility
Validation
npm --prefix scripts test(17 passed)node scripts/validate-registry.mjs(11 templates)out/index.html,out/404.html, 20 files)Companion platform and UI PRs will be linked after creation.
Summary by cubic
Classifies every catalog template with an immutable publishing compatibility contract and makes native templates buildable and gated by CI. Previously templates were static references; now
registry.jsondeclarespublishingCompatibility,buildProfile, and optionalrequiredCapabilities, and CI builds native templates and admits their production output.Migration
publishingCompatibility;buildProfileis required fornativeentries.requiredCapabilitiesonly for capabilities a template actually uses.Template changes
Written for commit 27144c1. Summary will update on new commits.