Skip to content

[codex] Keep inline top section title fragments together - #96

Open
gyx09212214-prog wants to merge 1 commit into
alphanome-ai:mainfrom
gyx09212214-prog:codex/inline-top-section-title
Open

[codex] Keep inline top section title fragments together#96
gyx09212214-prog wants to merge 1 commit into
alphanome-ai:mainfrom
gyx09212214-prog:codex/inline-top-section-title

Conversation

@gyx09212214-prog

Copy link
Copy Markdown

Summary

  • Keep inline top-section title fragments together when a parent element's full text already matches a top section and its direct children are inline text tags.
  • Add regression coverage for the MSFT PART I. FINANCIAL INFORMATION split described in Singular Visual Line Should Be Identified as a Single TextElement #78.
  • Preserve the existing behavior for block containers that include a top-section title plus body text.

Why this matters

Financial filing parsers should keep section headers as coherent semantic elements. Splitting PART I. FINANCIAL INFORMATION breaks downstream section navigation, table association, and agent retrieval over 10-Q filings.

Refs #78.

Validation

  • python -m py_compile sec_parser/processing_steps/individual_semantic_element_extractor/single_element_checks/top_section_title_check.py tests/unit/processing_steps/individual_semantic_element_extractor/single_element_checks/test_top_level_section_title_check.py
  • Direct TopSectionTitleCheck inline/block tests passed with dependency stubs.
  • git diff --check

Note: full pytest was not run locally because this environment does not have pytest installed.

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 7 complexity · 0 duplication

Metric Results
Complexity 7
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

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