Dev - #36
Merged
Merged
Dev#36
Conversation
…okens.bat) Guided browser wizard that pulls DeepSeek/Qwen auth tokens via a bookmarklet and a local 127.0.0.1-only server; .bat locates a Python 3.6+ interpreter and forwards extra args. 🤖
check_repo_guards() bans U+2014/U+2013 in docs/ text files; the copied script had 8 of them. Replaced with ASCII hyphens, no content changes. 🤖
- Python string literals: adjacent-literal concatenation (identical values) - Favicon / fonts / noise data-URIs: shared constants injected via the existing __key__ placeholder machinery (also fully percent-encodes the noise SVG, which previously mixed raw spaces with %20) - HTML: break between attributes, with a neutralizing comment between inline elements so no whitespace node is introduced - CSS: break between declarations inside rules Rendered setup/results pages and the bookmarklet are unchanged except for comment text and the percent-encoded noise URI. 🤖
…eassignment (ruff PLW0603) render_setup_page() and serve() now take the port as a parameter; the handler carries it as a class attribute set in serve(), so the setup page's bookmarklet still calls back on the effective --port. serve() also takes over the --no-browser handling from main(). Verified: ruff clean, and the setup/results pages + token collection work end-to-end on both the default port and a custom --port. 🤖
Purely mechanical: quote normalization, collapsed redundant parens, and a blank line after a local import. No behavior changes; snapshot comparison confirms TEXTS and the built bookmarklet are unchanged. 🤖
…uild (ruff PLW1510) The compiler probes intentionally tolerate failure (returncode is checked manually to try a fallback command), so check=False documents the existing behavior explicitly. No behavior change; unblocks the pre-existing 'ruff check .' CI failures on a clean dev checkout. 🤖
pylint cannot infer members on the sys.stdout TextIO wrapper, so read the encoding via getattr() + str() with an explicit truthiness guard. Behavior is identical (missing/None/empty encoding skips the re-wrap). 🤖
…launcher get_tokens.py / run_get_tokens.bat did not describe what the tool is; renamed to token_utility.py / token_utility.bat and updated the three stale self-references inside the script. token_utility.sh mirrors the .bat: probes python3/python and actually executes each candidate before accepting it, since some systems ship a python3 launcher that resolves but fails to run (127). Passes all arguments through to the Python script. Verified: both launchers run the script end-to-end, full test suite passes (ALL CHECKS PASSED). 🤖
Add standalone token utility
…subprocess check=False - Add incremental.finish() after aiter_bytes loop in _stream_openai and stream_openai so trailing SSE blocks without trailing newline are processed instead of dropped. - Fix PLW1510: add explicit check=False to subprocess.run calls in app.py.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull Request Template
Description
Brief description of changes.
Type of Change
Testing
Checklist