Skip to content

lookup: unwrap JSON-encoded TSV; 0.2.3 (AGG-117)#9

Merged
anthrax63 merged 1 commit into
mainfrom
agent/cto/09c2d050
Jul 2, 2026
Merged

lookup: unwrap JSON-encoded TSV; 0.2.3 (AGG-117)#9
anthrax63 merged 1 commit into
mainfrom
agent/cto/09c2d050

Conversation

@anthrax63

Copy link
Copy Markdown
Member

Summary

  • DDM `publish_to_session` (`type: json` + `type: extract` + `single_value: true`) writes the TSV as a JSON-encoded string. Verified on run 21525: `pipeline_runs.resultJson` = `"SUBS_KEY\tcg_beh_pd_1\t…\n7775108170\t0.1\t…\n"`.
  • `fetch_result` now unwraps that JSON layer before handing the payload to `parse_tsv`. Legacy raw-TSV payloads pass through unchanged.
  • Version bump 0.2.2 → 0.2.3 so the overlay tag moves off 0.2.2 without overwriting.

Test plan

  • `pytest tests/` — 53/53 pass, including new `test_fetch_result_unwraps_json_encoded_tsv` + `test_fetch_result_passes_through_plain_tsv`.
  • CI builds v0.2.3 image, overlay bump to v0.2.3, Argo sync.
  • `/single?iin=700205302324` returns JSON with real fields (`SUBS_KEY`, `cg_beh_pd_*`, `user_beh_score`, …), not `null`.

🤖 Generated with Claude Code

…AGG-117)

DDM's publish_to_session with `type: json` + `type: extract`
+ `single_value: true` writes the TSV payload as a JSON-encoded
string — one round of quoting/escaping over the actual TSV. Verified
today on run 21525 (systemId 7ba6e6b7-…): pipeline_runs.resultJson
= `"SUBS_KEY\tcg_beh_pd_1\t…\n7775108170\t0.1\t…\n"` — literal
double-quotes around escaped `\t`/`\n`, i.e. one JSON layer to peel.

fetch_result now json.loads() any resultJson that starts with a
double quote and returns the decoded string; anything else passes
through unchanged (defensive path for legacy raw TSV or dict results).

Bumps to 0.2.3 so the overlay can move off 0.2.2 without overwriting
that tag.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-authored-by: multica-agent <github@multica.ai>
@anthrax63 anthrax63 merged commit d45e9db into main Jul 2, 2026
3 checks passed
@anthrax63 anthrax63 deleted the agent/cto/09c2d050 branch July 2, 2026 08:41
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