feat: deprecate tracer path and migrate to opensre#1232
Conversation
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
Greptile SummaryThis PR migrates the codebase from the legacy Confidence Score: 5/5Safe to merge — all findings are P2 and the migration logic is well-guarded and tested. Only one P2 finding (hardcoded /tmp/opensre in a tool description string); no P1 or P0 issues. Migration path, sandbox restriction, permissions hardening, and keyring UX improvements are all correct and covered by tests. app/tools/run_diagnostic_code.py — hardcoded platform path in tool description Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[_load_raw called] --> B{STORE_PATH exists?}
B -- yes --> E[Load from ~/.opensre/integrations.json]
B -- no --> C{LEGACY_STORE_PATH exists?}
C -- no --> E2[Return empty v2 store]
C -- yes --> D{Test isolation guard
STORE_PATH patched but
LEGACY_STORE_PATH not?}
D -- yes --> E2
D -- no --> F[mkdir ~/.opensre]
F --> G[replace legacy → new path
os.replace]
G -- OSError --> H[copy bytes + unlink]
H -- OSError --> I[Log warning, return without migrating]
G --> J[chmod 0o600]
H --> J
J --> E
Reviews (2): Last reviewed commit: "fix: ci" | Re-trigger Greptile |
Apply ruff formatting to integrations store and extend the Mintlify Vale allowlist for domain-specific terms so quality and docs validation checks pass on this branch. Co-authored-by: Cursor <[email protected]>
Signed-off-by: vaibhav upreti <[email protected]>
…ecate_tracer_paths
Signed-off-by: vaibhav upreti <[email protected]>
|
@greptileai review |
|
⚡ LGTM → Merged. @VaibhavUpreti, your work is in. Every commit counts — thank you for this one. 👋 Join us on Discord - OpenSRE : hang out, contribute, or hunt for features and issues. Everyone's welcome. |

No description provided.