Skip to content

Phase 17: Antigravity plugin bundle skeleton#4

Merged
ProfRandom92 merged 6 commits into
mainfrom
phase-17-antigravity-plugin-bundle
Jun 6, 2026
Merged

Phase 17: Antigravity plugin bundle skeleton#4
ProfRandom92 merged 6 commits into
mainfrom
phase-17-antigravity-plugin-bundle

Conversation

@ProfRandom92
Copy link
Copy Markdown
Owner

Phase 17 implements the Antigravity Plugin Bundle Skeleton under the Git Safety Gate governance rules.

Scope:

  • ctxt antigravity command skeleton
  • Antigravity plugin templates
  • skills/hooks/MCP/permissions/subagents/rules template surfaces
  • Phase 17 documentation and report
  • no live hooks
  • no global install
  • no MCP server implementation
  • subagents advisory only
  • no LLM judge
  • no production/enterprise/compliance/certification claims
  • no cryptographic-proof claims

Copy link
Copy Markdown

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

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 implements the skeleton for the Antigravity plugin bundle (Phase 17), introducing the ctxt antigravity command with subcommands for exporting, validating skills, auditing hooks, and packaging plugins. It also adds local templates, documentation, and smoke tests for these features. The review feedback suggests using path.is_dir() instead of path.exists() to more robustly verify the local skills directory, and corrects a minor grammatical error in AGENTS.md.

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 thread src/cli.rs
("skills", Some("validate")) => {
println!("Validating repo-local skills...");
let path = std::path::Path::new("templates/antigravity/skills");
if path.exists() {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

Using path.is_dir() is more robust than path.exists() here because it ensures that the path is actually a directory before proceeding with the validation message, preventing false positives if a file with the same name exists.

Suggested change
if path.exists() {
if path.is_dir() {

Comment thread AGENTS.md
7. **Subagent Restrictions**: Subagents may validate, search, or inspect codebase assets but must never be used to bypass network, API key, browser, or write restrictions.
8. **Browser Sandbox**: Browser use is denied by default and requires explicit phase permission.
9. **Network Sandbox**: Network socket connections are denied by default and require explicit phase permission.
9. **Network Sandbox**: Network socket connections are denied by default and requires explicit phase permission.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

The verb should be plural ('require') to agree with the plural subject 'Network socket connections'.

Suggested change
9. **Network Sandbox**: Network socket connections are denied by default and requires explicit phase permission.
9. **Network Sandbox**: Network socket connections are denied by default and require explicit phase permission.

@ProfRandom92 ProfRandom92 merged commit 9bd2539 into main Jun 6, 2026
1 check passed
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.

1 participant