feat(cdsl): parse NPS holdings (holdings-only)#143
Conversation
Adds NPSScheme/NPSAccount to NSDLCASData.nps; resolves the NPS holding statement by order + units*nav. CDSL only; transactions out of scope.
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #143 +/- ##
==========================================
- Coverage 95.04% 94.97% -0.06%
==========================================
Files 19 19
Lines 2858 2979 +121
==========================================
+ Hits 2716 2829 +113
- Misses 142 150 +8 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
@mvineetmenon I've pushed a first draft of CDSL NPS support in this branch ( If you have uv installed, you could test it via uvx --from git+https://github.com/codereverser/casparser@feat/nps-cdsl casparser your_cdsl.pdf
# OR
uvx --from git+https://github.com/codereverser/casparser@feat/nps-cdsl casparser your_cdsl.pdf -o /tmp/cdsl_nps.jsonCan you please confirm if the numbers match your statement? |
|
@codereverser Congratuations and thanks for implementing this feature.
|
Handle page breaks.
|
Yes, the issue is fixed. Everything is getting parsed correctly. Thanks @codereverser |
|
Many thanks for testing. Merging this. NPS is CDSL holdings only now. Transaction support and NSDL NPS would be done separately. |


Adds support for NPS holdings (CDSL only)