Skip to content

fix(cdsl): recover split demat account summaries and broker names - #151

Merged
codereverser merged 2 commits into
codereverser:mainfrom
vmathur2004:fix/cdsl-demat-account-recovery
Sep 23, 2026
Merged

codereverser merged 2 commits into
codereverser:mainfrom
vmathur2004:fix/cdsl-demat-account-recovery

Conversation

@vmathur2004

Copy link
Copy Markdown
Contributor

Summary

  • reconstruct CDSL demat account summary rows when PDF extraction splits identifiers across four or five cells
  • join broker-name fragments before the DP and client identifiers so names do not collapse to a trailing word such as LIMITED
  • add regression tests for split account summaries and broker names

Validation

  • 107 focused demat unit tests passed
  • git diff --check passed

@codereverser

Copy link
Copy Markdown
Owner

Thanks for finding the issue and fixing it.

@codereverser
codereverser merged commit 34696d4 into codereverser:main Sep 23, 2026
0 of 3 checks passed
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.

2 participants