Simplify and fix: shared HTTP session, command table, Yahoo error handling, USD market caps, min_analysts, consolidation sweep by robdevops · Pull Request #37 · robdevops/finbot

CLAUDE.md: keep an open PR's title and description in step with every push

claude

committed

df5064c

Remove dead code: the discord and slack stubs, two uncalled util helpers, the ylim shim, unused imports

claude

committed

60c55ed

util: tables for markets, exchanges and ticker suffixes; one link builder; the chart setup written once

claude

committed

85c02c9

yahoo: quote fields read from tables; renewing the cookie and crumb is one thread at a time; fetch_each

claude

committed

5458ab6

webhook, sharesight, charts, bot, jobs: one copy of each repeated step

claude

committed

2607e89

reports: Yahoo details and compare histories fetched at the same time; one portfolio lookup and holding line

claude

committed

670910f

worker: each command matched where it is handled; typing runs as a with block; no print-and-reraise wrappers

claude

committed

92af627

net: one cookie-less HTTP session for every request; secrets kept out of the logs; a timeout on Yahoo's cookie

claude

committed

045be00

yahoo: a failed chart fetch raises; quotes parsed so that a missing, null or garbled field is left out

claude

committed

c695ac8

Reports go where they should from one place; price and performance split into gather, report and delivery

claude

committed

9495b1b

worker: one function per command, in an ordered table; no interim notices; .hello no longer grows its word list

claude

committed

aaf8e14

Markets: TSE is Tokyo and TSX Toronto, as Sharesight has them; NZX and SGX get their Yahoo suffixes, flags and currencies

claude

committed

d2b297c

Remove the commented-out code; milestone records a ticker only once it has both new values

claude

committed

074c089

Profile: a field Yahoo did not send leaves its line out instead of crashing; data files read in one place; unused imports gone

claude

committed

d1a6e12

Telegram calls share telegram.call; transform_title reads its replacements from two ordered tables

claude

committed

e990027

cal takes one event; one portfolio lookup answers an unknown name the same way everywhere; trades loses its dead lines

claude

committed

ef0d4c3

rating keeps what it has recorded; podcasts survives feeds with undated items and keeps its state like the other scripts

claude

committed

9abb58e

yahoo.fetch_details: one fetch of many tickers' details, where six loops did it; the ratings list sorts once

claude

committed

7441bfc

A company profile keeps the end of its description; charts share their axis styling and heading; config errors print in one place

claude

committed

cfb9a81

shorts: the list and its chart come from one sorted list

claude

committed

3a7039c

Record highs are all-time; ratings list the strongest first; .top is gone; a bare number on .price is a percent

claude

committed

b89effc

.buy/.sell need 7 analysts; market caps compare in US dollars, so a won market cap no longer tops .marketcap

claude

committed

21213a0

Keyboard: one .buy & sell button sends both rating reports; .forward pe takes the spare place; buttons stay alphabetical

claude

committed

8024454

.forward pe (and its keyboard button) sends a bar chart, as .peg does

claude

committed

abe262d

min_analysts: one preference for .buy, .sell and the forward P/E lists (default 7), documented

claude

committed

07b4f43

Merge remote-tracking branch 'origin/main' into claude/kind-newton-ad0z2q

claude

committed

34bc506

Navigation Menu

Repository navigation

Simplify and fix: shared HTTP session, command table, Yahoo error handling, USD market caps, min_analysts, consolidation sweep - #37

Simplify and fix: shared HTTP session, command table, Yahoo error handling, USD market caps, min_analysts, consolidation sweep#37
robdevops merged 26 commits into
mainfrom
claude/kind-newton-ad0z2q

Commits on Oct 14, 2026

CLAUDE.md: keep an open PR's title and description in step with every push

Remove dead code: the discord and slack stubs, two uncalled util helpers, the ylim shim, unused imports

util: tables for markets, exchanges and ticker suffixes; one link builder; the chart setup written once

yahoo: quote fields read from tables; renewing the cookie and crumb is one thread at a time; fetch_each

webhook, sharesight, charts, bot, jobs: one copy of each repeated step

reports: Yahoo details and compare histories fetched at the same time; one portfolio lookup and holding line

worker: each command matched where it is handled; typing runs as a with block; no print-and-reraise wrappers

net: one cookie-less HTTP session for every request; secrets kept out of the logs; a timeout on Yahoo's cookie

yahoo: a failed chart fetch raises; quotes parsed so that a missing, null or garbled field is left out

Reports go where they should from one place; price and performance split into gather, report and delivery

worker: one function per command, in an ordered table; no interim notices; .hello no longer grows its word list

Markets: TSE is Tokyo and TSX Toronto, as Sharesight has them; NZX and SGX get their Yahoo suffixes, flags and currencies

Remove the commented-out code; milestone records a ticker only once it has both new values

Profile: a field Yahoo did not send leaves its line out instead of crashing; data files read in one place; unused imports gone

Telegram calls share telegram.call; transform_title reads its replacements from two ordered tables

cal takes one event; one portfolio lookup answers an unknown name the same way everywhere; trades loses its dead lines

rating keeps what it has recorded; podcasts survives feeds with undated items and keeps its state like the other scripts

yahoo.fetch_details: one fetch of many tickers' details, where six loops did it; the ratings list sorts once

A company profile keeps the end of its description; charts share their axis styling and heading; config errors print in one place

shorts: the list and its chart come from one sorted list

Record highs are all-time; ratings list the strongest first; .top is gone; a bare number on .price is a percent

.buy/.sell need 7 analysts; market caps compare in US dollars, so a won market cap no longer tops .marketcap

Keyboard: one .buy & sell button sends both rating reports; .forward pe takes the spare place; buttons stay alphabetical

.forward pe (and its keyboard button) sends a bar chart, as .peg does

min_analysts: one preference for .buy, .sell and the forward P/E lists (default 7), documented

Merge remote-tracking branch 'origin/main' into claude/kind-newton-ad0z2q