Clean/integrated low risk 20260628 - #10
Merged
Merged
Conversation
|
Important Review skippedToo many files! This PR contains 324 files, which is 174 over the limit of 150. To get a review, narrow the scope: Upgrade to a paid plan to raise the limit. ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (561)
📒 Files selected for processing (324)
You can disable this status message by setting the Use the checkbox below for a quick retry:
|
|
✅ Health: 6.3 → 6.4 (+0.1) 🚨 Change risk: 9.7/10 (high)
🔥 Hotspots touched (3)
🔗 Hidden coupling (3 files)
💀 Dead code (4 findings)
1 more
👀 Suggested reviewers @Curry @2233admin 📊 Full report · ⭐ Star Repowise · 📥 Install bot · Last updated 2026-06-28 10:28 UTC |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Code Review
This pull request shifts the project's positioning from a terminal-first toolkit to a conversation-first 'Chat Arts' visual language, updating the documentation in README.md, SKILL.md, and a new CONTEXT.md file. It also adds a .deepsource.toml configuration, updates .gitignore, and removes various research-related files. Feedback suggests correcting the DeepSource Python runtime version to '3.x' and updating the total chart type count in the README to '79' to accurately reflect the newly added types.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
| enabled = true | ||
|
|
||
| [analyzers.meta] | ||
| runtime_version = "3.x.x" |
Contributor
There was a problem hiding this comment.
DeepSource's Python analyzer expects the runtime_version to be a major.minor version (e.g., 3.10, 3.11) or the wildcard 3.x. The value 3.x.x is not a standard supported version format and may cause configuration parsing issues.
Suggested change
| runtime_version = "3.x.x" | |
| runtime_version = "3.x" |
| | media *(image uses Pillow/chafa; video requires chafa + ffmpeg)* | `image` `video` | | ||
|
|
||
| Total: **74 types**. See `CHART_TYPE_COUNT` in `cli_charts/chart.py` for the authoritative count. | ||
| Total: **76 types**. See `CHART_TYPE_COUNT` in `cli_charts/chart.py` for the authoritative count. |
Contributor
There was a problem hiding this comment.
The total count of chart types is stated as 76 types, but 5 new types (to-drawio, fonts, chat-health, petiglyph, calibrate) have been added to the misc row (increasing it from 21 to 26 types). Since the previous total was 74 types, the new total should be updated to 79 types to maintain consistency.
Suggested change:
Total: **79 types**. See `CHART_TYPE_COUNT` in `cli_charts/chart.py` for the authoritative count.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
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
1 participant
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
Linear
Closes XAR-
Test plan
npx nx run glyph-arts:testnpx nx run glyph-arts:lintnpx nx run glyph-arts:doctorNotes for reviewer