docs(cesium-context7): improve skill description, restructure content, and add evals#24
Open
MKartaviciute wants to merge 3 commits into
Open
docs(cesium-context7): improve skill description, restructure content, and add evals#24MKartaviciute wants to merge 3 commits into
MKartaviciute wants to merge 3 commits into
Conversation
jdehorty
reviewed
Mar 24, 2026
Contributor
There was a problem hiding this comment.
Is there supposed to be a harness for this file? How are you running this?
jdehorty
reviewed
Mar 24, 2026
| { | ||
| "id": 3, | ||
| "prompt": "I'm using Cesium for Unreal Engine 5. I have a Blueprint actor that I want to always stay at a fixed lat/lon on the globe, even as the player moves around. I tried setting the actor's world position but it drifts. What's the correct way to do this?", | ||
| "expected_output": "Guidance on using ACesiumGlobeAnchor (Globe Anchor) component to lock a Blueprint actor to a geographic coordinate, based on fetched docs from /cesiumgs/cesium-unreal.", |
Contributor
There was a problem hiding this comment.
Is ACesiumGlobeAnchor a hallucinated class name? Where did this come from?
jdehorty
reviewed
Mar 24, 2026
| @@ -1,73 +0,0 @@ | |||
| --- | |||
Contributor
There was a problem hiding this comment.
The PR removes .github/skills/cesium-context7/SKILL.md, but the README still tells users to configure that exact path for GitHub Copilot.
Contributor
Author
There was a problem hiding this comment.
It is under workspace specific recommendations, so I don't think it is a problem. The issue is that it is quite difficult to main the same content in 2 places
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
3 participants
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.
Description
This pull request significantly revises the Cesium Context7 skill documentation to clarify usage guidelines, improve workflow instructions, and add example queries. It also introduces a new evaluation file to test the skill's behavior across common Cesium-related scenarios. The most important changes are grouped below:
Documentation overhaul and workflow improvements:
SKILL.mdfile forcesium-context7has been rewritten to clarify when and how to use the skill, emphasize always fetching documentation before writing or reviewing code, and provide a concise workflow with example queries for CesiumJS, Unreal, Unity, and 3D Tiles. It now includes a table of library IDs and query tips for better results..github/skills/cesium-context7/SKILL.mdversion has been removed, eliminating redundant and less clear instructions.Evaluation and testing additions:
evals.jsonfile has been added underskills/cesium-context7/evals/, containing five realistic evaluation scenarios. These cover CesiumJS entity placement, 3D Tiles refinement, Unreal Engine globe anchoring, CesiumJS camera animation, and feature styling, each with expectations for correct tool usage and accurate, documentation-based answers.Issue number and link
Testing plan
Author checklist