Mark Antigravity as intentionally undocumented and gate engine-doc drift checks on that intent#45018
Conversation
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
🤖 PR Triage
Rationale: Adds
|
|
Great work here, The changes are well-scoped:
|
🤖 PR Triage
Rationale: Draft. Marks Antigravity as intentionally undocumented, preventing doc-healer from repeatedly reproposing rejected changes. Carry-over from prior run. Low risk, 8 files. Triage run §29183606049
|
There was a problem hiding this comment.
Pull request overview
Marks Antigravity as intentionally undocumented and prevents automated drift workflows from proposing documentation/schema synchronization.
Changes:
- Added engine-level documentation intent and registry filtering.
- Marked Antigravity undocumented with regression coverage.
- Updated and recompiled drift-analysis workflows.
Show a summary per file
| File | Description |
|---|---|
pkg/workflow/antigravity_engine.go |
Marks Antigravity undocumented. |
pkg/workflow/antigravity_engine_test.go |
Verifies Antigravity’s intent. |
pkg/workflow/agentic_engine.go |
Adds introspection and documented-engine filtering. |
pkg/workflow/agentic_engine_test.go |
Tests registry filtering. |
.github/workflows/schema-consistency-checker.md |
Excludes intentionally undocumented engines. |
.github/workflows/schema-consistency-checker.lock.yml |
Updates compiled workflow metadata. |
.github/workflows/deep-report.md |
Suppresses intentional engine-doc drift tasks. |
.github/workflows/deep-report.lock.yml |
Updates compiled workflow metadata. |
Review details
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 8/8 changed files
- Comments generated: 0
- Review effort level: Medium
|
@copilot please run the Run: https://github.com/github/gh-aw/actions/runs/29192100649
|
# Conflicts: # .github/workflows/deep-report.lock.yml # .github/workflows/schema-consistency-checker.lock.yml Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>
🤖 PR Triage
Gates engine-doc drift checks on Antigravity intent flag. Small change (+42/-2). Review in batch.
|
Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>
Done. The branch was behind |
|
@copilot please run the Run: https://github.com/github/gh-aw/actions/runs/29197331019
|
Branch is up to date with |
|
@copilot please run the Run: https://github.com/github/gh-aw/actions/runs/29199165277
|
Branch is up to date with |
Doc-healer/DDUw repeatedly proposed the same antigravity docs/schema sync despite three maintainer rejections. This change records that exclusion in code and teaches drift-analysis workflows to treat it as intentional.
Engine intent model: explicit undocumented flag
undocumented booltoBaseEngine.IsUndocumented()for runtime introspection.EngineRegistry.GetDocumentedEngines()to return only engines that should participate in docs/schema parity checks.Antigravity marked as intentionally excluded
undocumented: trueinNewAntigravityEngine()so antigravity remains registered/usable but is not considered a docs/schema drift target.Drift-detection workflow guidance updated
schema-consistency-checker.mdanddeep-report.mdinstructions to skip docs/schema sync findings for engines explicitly marked undocumented..lock.ymlworkflow artifacts.Focused regression coverage
IsUndocumented() == true.GetDocumentedEngines()excludesantigravityand keeps documented engines (e.g.pi).