fix: repair botched market-report merge (conflict markers + garbled HTML) #192

Merged

joshtronic merged 1 commit from fix/market-report-merge-resolution into master

2026-06-10 00:30:12 +00:00

joshtronic commented

2026-06-10 00:29:33 +00:00

Owner

PR #191 merged an unresolved conflict: market_render_markdown still
carried <<<<<<< / ======= / >>>>>>> markers, and market_render_html had
a stray line spliced into the row. jq failed to compile,
so every weekday market tick errored ("syntax error, unexpected '+'")
and no report email went out.

Resolve by combining both sides' intent: drop the Company column (#189)
and keep Previous Low before Previous High (#190) across the markdown
and HTML tables. Verified jq compiles and both renderers emit five
matching columns against a fixture.

Co-Authored-By: Claude Opus 4.8 (1M context) noreply@anthropic.com

PR #191 merged an unresolved conflict: market_render_markdown still carried <<<<<<< / ======= / >>>>>>> markers, and market_render_html had a stray <thead> line spliced into the <tbody> row. jq failed to compile, so every weekday market tick errored ("syntax error, unexpected '+'") and no report email went out. Resolve by combining both sides' intent: drop the Company column (#189) and keep Previous Low before Previous High (#190) across the markdown and HTML tables. Verified jq compiles and both renderers emit five matching columns against a fixture. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

joshtronic added 1 commit

2026-06-10 00:29:33 +00:00

fix: repair botched market-report merge (conflict markers + garbled HTML)

Lint / check-sync (push) Successful in 3s

Details

Lint / check-sync (pull_request) Successful in 3s

Details

a526881216

PR #191 merged an unresolved conflict: market_render_markdown still
carried <<<<<<< / ======= / >>>>>>> markers, and market_render_html had
a stray <thead> line spliced into the <tbody> row. jq failed to compile,
so every weekday market tick errored ("syntax error, unexpected '+'")
and no report email went out.

Resolve by combining both sides' intent: drop the Company column (#189)
and keep Previous Low before Previous High (#190) across the markdown
and HTML tables. Verified jq compiles and both renderers emit five
matching columns against a fixture.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

joshtronic merged commit 98bfd4b3e0 into master

2026-06-10 00:30:12 +00:00

joshtronic deleted branch fix/market-report-merge-resolution

2026-06-10 00:30:12 +00:00

joshtronic referenced this pull request from a commit

2026-06-10 00:30:13 +00:00

Merge pull request 'fix: repair botched market-report merge (conflict markers + garbled HTML)' (#192) from fix/market-report-merge-resolution into master