Skip to content

Triage open bug reports (#5, #11, #16)#19

Open
rar-file wants to merge 1 commit into
mainfrom
fix/triage-bugs-summarize-research-webflag
Open

Triage open bug reports (#5, #11, #16)#19
rar-file wants to merge 1 commit into
mainfrom
fix/triage-bugs-summarize-research-webflag

Conversation

@rar-file

Copy link
Copy Markdown
Owner

Three small, mechanical fixes to clear out the open issue tracker.

#11/summarize crash. self.history doesn't exist on SurfCLI; switched to self.config.messages (what every other handler uses).

#16do_research literal \n. Source contained doubled backslashes, so the model received one smushed line and the live-stream splitter never matched. Collapsed the affected lines inside do_research to real \n.

#5--web flag. README and setup.sh documented python chat.py --web, but the flag wasn't registered. Added --web [PORT] to argparse; when set, it imports web_ui.launch(), runs it, and exits before the chat loop.

Verified python3 chat.py --help shows the flag and py_compile is clean.

- /summarize: use self.config.messages (not the nonexistent self.history). Closes #11.
- /research: collapse literal '\\n' sequences to real newlines so the
  research prompt isn't sent as one smushed line and live-streaming
  splits on real newlines. Closes #16.
- main: add --web [PORT] argparse arg that launches web_ui.launch() and
  exits, matching the README/setup.sh docs. Closes #5.
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