Skip to content

fix old description#170

Open
hheei wants to merge 2 commits into
cortexkit:mainfrom
hheei:patch-1
Open

fix old description#170
hheei wants to merge 2 commits into
cortexkit:mainfrom
hheei:patch-1

Conversation

@hheei

@hheei hheei commented Jul 25, 2026

Copy link
Copy Markdown

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.


Summary by cubic

Update the pi-plugin README to point AFT config to the new CortexKit locations and remove unsupported .json variants. Docs now reference ~/.config/cortexkit/aft.jsonc and <project>/.cortexkit/aft.jsonc, replacing ~/.pi/agent and .pi.

Written for commit a160ba9. Summary will update on new commits.

Review in cubic

Greptile Summary

Updates the Pi plugin documentation to list only the supported user and project aft.jsonc configuration paths.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
packages/pi-plugin/README.md Removes the unsupported .json alternatives and replaces the obsolete Pi-specific locations with the current CortexKit aft.jsonc paths.

Reviews (2): Last reviewed commit: "Update README.md" | Re-trigger Greptile

Comment thread packages/pi-plugin/README.md Outdated
Comment on lines +62 to +63
- **User:** `~/.config/cortexkit/aft.jsonc` (or `.json`)
- **Project:** `<project>/.cortexkit/aft.jsonc` (or `.json`)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Unsupported JSON config paths

When a user follows these instructions and creates either documented aft.json variant, the plugin checks only the corresponding aft.jsonc path, causing the user configuration or project overrides to be silently ignored.

@cubic-dev-ai cubic-dev-ai Bot 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.

No issues found across 1 file

Re-trigger cubic

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