Skip to content

smoke: fork write flow validation #22

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
keithah wants to merge 1 commit into from
Closed

smoke: fork write flow validation #22

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions .kodiai.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,3 +1,11 @@
review:
uiRereviewTeam: aireview
requestUiRereviewTeamOnOpen: true

mention:
enabled: true

write:
enabled: true
allowPaths:
Comment on lines +8 to +10
Copy link

Copilot AI Feb 11, 2026

Choose a reason for hiding this comment

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

Enabling write.enabled: true on the default branch allows mention-driven commits to land automatically (within the allowPaths policy). If this PR is only meant for a temporary smoke validation, consider keeping write-mode disabled on main and enabling it only on the smoke branch/PR under test, or time-boxing/reverting it afterward to reduce ongoing operational/security exposure.

Copilot uses AI. Check for mistakes.
- docs/**
Copy link
Copy Markdown

@kodiai kodiai Bot Feb 11, 2026

Choose a reason for hiding this comment

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

The allowPaths pattern docs/** will match any path starting with docs/, but it's worth confirming this is the intended behavior. If the intent is to only allow modifications within the docs directory and its subdirectories, this pattern is correct. However, if there are specific subdirectories within docs/ that should be excluded (e.g., docs/internal/ or sensitive documentation), those should be explicitly denied or this pattern should be more restrictive.

Copy link

Copilot AI Feb 11, 2026

Choose a reason for hiding this comment

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

write.allowPaths is currently set to docs/**, which permits write-mode changes anywhere under docs/. For a smoke test, consider narrowing this to the minimal scope needed (e.g., docs/smoke/** or specific files) to reduce the blast radius if someone triggers @kodiai apply: unintentionally.

Suggested change
- docs/**
- docs/smoke/**

Copilot uses AI. Check for mistakes.
2 changes: 2 additions & 0 deletions docs/smoke/xbmc-kodiai-write-flow.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
Smoke Test: xbmc/kodiai write-mode end-to-end

Seed note: fork smoke branch baseline (2026-02-11).

Goal

Validate the full mention-driven write flow in the default repo (`xbmc/kodiai`) using:
Expand Down