Skip to content

fix: demo stability + remove committed tokens#6

Closed
youssofal wants to merge 2 commits into
mainfrom
codex/m8-stability
Closed

fix: demo stability + remove committed tokens#6
youssofal wants to merge 2 commits into
mainfrom
codex/m8-stability

Conversation

@youssofal
Copy link
Copy Markdown
Owner

Summary:

  • Remove committed plaintext tokens/keys from HANDOFF.md
  • Prevent Smart/Autopilot deadlocks by clearing isBusy on Fast Path early returns
  • Avoid native destroyed-mutex crashes by not destroying CardClassifier during Activity teardown

Notes:

  • This PR is intentionally minimal and should not change core behavior when the happy-path succeeds.
  • Please rotate any keys/tokens that were previously committed.

Tested:

  • Not built in this environment (Android SDK missing). Needs device verification on Samsung A35.

- Remove plaintext API tokens/keys from HANDOFF.md and replace with safe guidance

Tested: N/A (docs only)
Device: N/A
- Clear isBusy on all Fast Path early-returns so Smart/Autopilot can't deadlock on races
- Stop calling CardClassifier.destroy() during Activity teardown to avoid native destroyed-mutex crashes

Tested: Not run here (Android SDK not installed in this environment)
Device: Samsung Galaxy A35, Android 16, 1080x2340
@youssofal
Copy link
Copy Markdown
Owner Author

Main now removed HANDOFF.md (commit 8b66750), so this PR’s docs change is obsolete and likely to conflict. I opened a replacement PR with the same stability fixes rebased on latest main: PR #7 (head codex/m8-stability-v2). Recommend closing this PR and merging #7 instead.

@youssofal youssofal closed this Feb 8, 2026
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