Skip to content

Remove committed Claude config secrets and fix custom orchestrator/setup scripts#9

Draft
Claude wants to merge 1 commit into
PROBLEMfrom
claude/check-all-my-problem
Draft

Remove committed Claude config secrets and fix custom orchestrator/setup scripts#9
Claude wants to merge 1 commit into
PROBLEMfrom
claude/check-all-my-problem

Conversation

@Claude

@Claude Claude AI commented Jun 19, 2026

Copy link
Copy Markdown

The repository contained a committed dump of ~/.claude.json (the PROBLEM ENCOUNTER file) exposing a live MCP API key and account PII, alongside several custom helper files in idée/ and Script/ with latent bugs.

Note: these changes are proposed/draft. The exposed x-mcp-key must still be rotated independently — removing the file does not purge it from git history.

Secrets & PII

  • Removed PROBLEM ENCOUNTER (full ~/.claude.json dump) containing a literal x-mcp-key, OAuth account email/UUIDs, and userID/machineID fingerprints.
  • Added .gitignore entries for *.claude.json dumps and exported logs to prevent recurrence.

idée/setup.sh

  • Corrected the smoke-test path so it resolves the workflow YAMLs in the same directory instead of a non-existent workflows/ subdir.

Script/run_devcontainer_claude_code.ps1

  • Set $ErrorActionPreference = 'Stop' (and explicit $LASTEXITCODE checks) so failures from native docker/podman/devcontainer calls are actually caught instead of silently passing.

idée/workflow_orchestrator.py

  • Rewrote the in-degree/topological-sort logic using standard Kahn's algorithm for robust cycle detection on arbitrary graphs.
  • Treat retry: 0 as an explicit error rather than silently skipping the agent.

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.

2 participants