Skip to content

chore: skill content update#93

Merged
lxfu1 merged 2 commits into
feat-context-servicefrom
chore/update-skill-retrieval-method
Jul 2, 2026
Merged

chore: skill content update#93
lxfu1 merged 2 commits into
feat-context-servicefrom
chore/update-skill-retrieval-method

Conversation

@lxfu1

@lxfu1 lxfu1 commented Jul 1, 2026

Copy link
Copy Markdown
Member

No description provided.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the documentation for AntV G2, G6, and X6 skills by replacing the POST-based content retrieval endpoints with a new GET-based /api/v1/context/retrieve endpoint, updating the quick reference query tables, and adding CDN usage examples. The review feedback identifies critical issues in the newly added CDN examples, including a missing node definition and constructor inconsistency in the G6 graph example, as well as an incorrect library version and script path in the X6 editor example.

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.

Comment on lines +33 to +42
const graph = new G6.Graph({
container: 'container',
data: {
nodes: [{ id: 'node-1', style: { labelText: 'Node 1' } }],
edges: [{ source: 'node-1', target: 'node-2' }],
},
layout: { type: 'force' },
behaviors: ['drag-canvas', 'zoom-canvas', 'drag-element'],
});
graph.render();

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

There are two issues in this CDN usage example:

  1. Missing Node: The edges array references node-2 as a target, but node-2 is not defined in the nodes array. This will cause rendering or data integrity issues in G6.
  2. Constructor Consistency: The code uses new G6.Graph(...), which contradicts the Critical Rule defined later in this file: "MUST: Use new Graph({...}) — NOT v4 new G6.Graph()". Destructuring Graph from G6 first allows using new Graph(...) consistently.
Suggested change
const graph = new G6.Graph({
container: 'container',
data: {
nodes: [{ id: 'node-1', style: { labelText: 'Node 1' } }],
edges: [{ source: 'node-1', target: 'node-2' }],
},
layout: { type: 'force' },
behaviors: ['drag-canvas', 'zoom-canvas', 'drag-element'],
});
graph.render();
const { Graph } = G6;
const graph = new Graph({
container: 'container',
data: {
nodes: [
{ id: 'node-1', style: { labelText: 'Node 1' } },
{ id: 'node-2', style: { labelText: 'Node 2' } },
],
edges: [{ source: 'node-1', target: 'node-2' }],
},
layout: { type: 'force' },
behaviors: ['drag-canvas', 'zoom-canvas', 'drag-element'],
});
graph.render();

Comment thread skills/antv-x6-editor/SKILL.md Outdated
### CDN Usage

```html
<script src="https://unpkg.com/@antv/x6@2/dist/x6.js"></script>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

This file is dedicated to X6 v3 (as indicated by the title # X6 v3 Graph Editor). However, the CDN script tag is loading @antv/x6@2 (v2) and pointing to /dist/x6.js (which was the path format for v1, whereas v2 and v3 use /dist/index.js).

To ensure compatibility with X6 v3 features and avoid loading the wrong version, the CDN URL should be updated to point to @antv/x6@3/dist/index.js.

Suggested change
<script src="https://unpkg.com/@antv/x6@2/dist/x6.js"></script>
<script src="https://unpkg.com/@antv/x6@3/dist/index.js"></script>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the AntV G2/G6/X6 skill documentation to include CDN usage snippets and to refresh the “Content Retrieval” instructions/quick-reference queries.

Changes:

  • Added “CDN Usage” HTML snippets for G2, G6, and X6 SKILL docs.
  • Replaced the older POST-based content-retrieval guidance with a GET-based /api/v1/context/retrieve section and updated quick reference tables accordingly.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.

File Description
skills/antv-x6-editor/SKILL.md Adds CDN snippet and rewrites content retrieval + repositions Quick Reference table.
skills/antv-g6-graph/SKILL.md Adds CDN snippet and rewrites content retrieval + repositions Quick Reference table.
skills/antv-g2-chart/SKILL.md Adds CDN snippet and rewrites content retrieval + repositions Quick Reference table.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +31 to +43
<script src="https://unpkg.com/@antv/g6@5/dist/g6.min.js"></script>
<script>
const graph = new G6.Graph({
container: 'container',
data: {
nodes: [{ id: 'node-1', style: { labelText: 'Node 1' } }],
edges: [{ source: 'node-1', target: 'node-2' }],
},
layout: { type: 'force' },
behaviors: ['drag-canvas', 'zoom-canvas', 'drag-element'],
});
graph.render();
</script>
Comment thread skills/antv-g6-graph/SKILL.md Outdated
Comment thread skills/antv-g2-chart/SKILL.md Outdated
Comment thread skills/antv-x6-editor/SKILL.md Outdated
Comment thread skills/antv-x6-editor/SKILL.md Outdated
@lxfu1
lxfu1 merged commit 782409c into feat-context-service Jul 2, 2026
4 checks passed
@lxfu1
lxfu1 deleted the chore/update-skill-retrieval-method branch July 2, 2026 01:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants