Skip to content

fix(pairing): accept spaced approval codes - #89937

Open
teknium1 wants to merge 1 commit into
mainfrom
nanoclaw-port/pairing-code-whitespace
Open

fix(pairing): accept spaced approval codes#89937
teknium1 wants to merge 1 commit into
mainfrom
nanoclaw-port/pairing-code-whitespace

Conversation

@teknium1

Copy link
Copy Markdown
Contributor

Summary

Hermes pairing approval now accepts codes pasted with visual whitespace while still rejecting chatter-wrapped text.

Changes

  • gateway/pairing.py: normalize internal whitespace before matching pairing approval codes.
  • tests/gateway/test_pairing.py: cover spaced-code acceptance and word-wrapped rejection.

Source

Validation

Check Result
scripts/run_tests.sh tests/gateway/test_pairing.py -q 31 passed
Isolated E2E with temp HERMES_HOME Spaced code accepted; chatter-wrapped code rejected

Infographic

Skipped: FAL image generation returned Exhausted balance during this cron run.

Port from nanocoai/nanoclaw#3282: normalize visual whitespace in DM pairing codes while preserving exact-match rejection for chatter-wrapped input.
@github-actions

github-actions Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

૮ >ﻌ< ა ci review

ran on 127a7ab — fix(pairing): accept spaced approval codes

❌ Job failures

Python tests / Run tests slice 12/12 · View job

Job Python tests / Run tests slice 12/12 failed.


⚠️ Warnings

OSV vulnerability scan · View job

7 known vulnerabilities found in pinned dependencies.

How to fix:

Review the findings in the Security tab. Update the affected dependencies if a patched version is available.


debug info

CI timings

CI timings · View report · View job

Wall time 4m33s vs 4m4s (+11.9%). 10 job(s) slower, 13 faster, 1 unchanged.

  • Python tests / Run tests slice 3/12: +46.0s
  • Python tests / Run tests slice 11/12: -39.0s
  • Python tests / Run tests slice 2/12: +30.0s
  • Python tests / Run tests slice 12/12: +30.0s
  • OSV scan / Scan lockfiles / osv-scan: -21.0s

@alt-glitch alt-glitch added type/bug Something isn't working comp/gateway Gateway runner, session dispatch, delivery area/auth Authentication, OAuth, credential pools P2 Medium — degraded but workaround exists sweeper:risk-security-boundary Sweeper risk: may affect sandboxing, auth, credentials, or sensitive data labels Aug 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/auth Authentication, OAuth, credential pools comp/gateway Gateway runner, session dispatch, delivery P2 Medium — degraded but workaround exists sweeper:risk-security-boundary Sweeper risk: may affect sandboxing, auth, credentials, or sensitive data type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants