Conversation
There was a problem hiding this comment.
Pull request overview
This PR upgrades project dependencies and adds comprehensive documentation for LLM agents working with the Respo framework and Calcit language.
Key changes:
- Dependency upgrades:
@calcit/procs(0.9.20 → 0.10.4),vite(7.1.10 → 7.3.0),respo.calcit(0.16.21 → 0.16.23), and related esbuild/rollup packages - Version bump: Project version updated from 0.10.2 to 0.10.3
- Documentation additions: 920 lines of LLM agent development guide for Respo, 456 lines of Calcit CLI reference guide
- Code enhancements: Added
:examplesfields to all code definitions and improved documentation strings
Reviewed changes
Copilot reviewed 6 out of 9 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| package.json | Updated vite to 7.3.0 and @calcit/procs to 0.10.4 |
| yarn.lock | Updated lockfile with new dependency versions for esbuild (0.25.10→0.27.2), rollup (4.52.4→4.54.0), vite (7.1.10→7.3.0), and min-document (2.19.0→2.19.2) |
| deps.cirru | Updated Calcit version to 0.10.4, respo.calcit to 0.16.23, and memof to explicit version 0.0.17 |
| compact.cirru | Added comprehensive documentation strings and examples to all component definitions; bumped version to 0.10.3; added empty :examples arrays throughout |
| llms/Respo.md | New 920-line comprehensive guide for LLM agents on Respo development, including CLI commands, patterns, debugging, and best practices |
| Agents.md | New 456-line Calcit CLI reference guide covering all query, edit, tree, and documentation commands |
| .gitignore | Added .DS_Store to ignore list |
| .gitattributes | Marked Agents.md and llms/*.md as linguist-generated to exclude from GitHub language stats |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| :code $ quote | ||
| defstyle style-trigger $ {} | ||
| "\"&" $ {} (:border-radius "\"50%") (:position :absolute) (:transform "\"translate(-50%,-50%)") (:top "\"50%") (:left "\"50%") (:width 0) (:height 0) (:transition-duration "\"300ms") (:transition-delay "\"100ms") (:pointer-events :none) (:z-index 900) (:opacity 1) | ||
| "\"&" $ {} (:border-radius "\"50%") (:position :absolute) (:transform "\"translate(-50%,-50%)") (:top "\"50%") (:left "\"50%") (:width 0) (:height 0) (:transition-duration "\"300ms") (:transition-delay "\"100ms") (:pointer-events :none) (:z-index |900) (:opacity 1) |
There was a problem hiding this comment.
The z-index value has been changed from the number 900 to the string |900. In CSS properties, z-index should be a numeric value, not a string. This change may cause the z-index style to be invalid or not work as expected. Consider reverting this to the numeric value 900.
| :background-color $ hsl 0 30 10 0.6 | ||
| :position :fixed | ||
| :z-index 999 | ||
| :z-index |999 |
There was a problem hiding this comment.
The z-index value has been changed from the number 999 to the string |999. In CSS properties, z-index should be a numeric value, not a string. This change may cause the z-index style to be invalid or not work as expected. Consider reverting this to the numeric value 999.
| :z-index |999 | |
| :z-index 999 |
| def site $ {} (:dev-ui "\"http://localhost:8100/main-fonts.css") (:release-ui "\"http://cdn.tiye.me/favored-fonts/main-fonts.css") (:cdn-url "\"http://cdn.tiye.me/calcit-workflow/") (:title "\"Alerts") (:icon "\"http://cdn.tiye.me/logo/respo.png") (:storage-key "\"respo-alerts") | ||
| :examples $ [] |
There was a problem hiding this comment.
In def site, the :release-ui, :cdn-url, and :icon fields are hard-coded to http:// CDN URLs, meaning the application will load CSS and assets over an unencrypted connection. A network attacker on the path can tamper with these responses to inject malicious CSS or modified assets into your UI. Switch these URLs to https:// equivalents (or serve them from a trusted HTTPS origin) so that all external assets are fetched over TLS with integrity protection.
2 participants
No description provided.