Skip to content

Preserve code blocks and indentation inside Markdown lists - #161

Merged
ralyodio merged 2 commits into
profullstack:masterfrom
RowletCC:fix/list-content-indent
Sep 21, 2026
Merged

ralyodio merged 2 commits into
profullstack:masterfrom
RowletCC:fix/list-content-indent

Conversation

@RowletCC

@RowletCC RowletCC commented Sep 20, 2026

Copy link
Copy Markdown
Contributor

List continuation lines currently lose only two characters beyond the list indentation, regardless of the marker width. As a result, even a 1. item adds an unwanted space to every line of its fenced code; longer numeric markers leave enough indentation that the fence is rendered as ordinary text.

This change records the prefix consumed for each item and removes that prefix from its continuation lines. It preserves indentation inside code and recalculates the prefix for subsequent items, including a change from 9999. to 10000.. Nested lists continue to parse recursively. Blank lines followed by content at the item's content indentation remain inside the item, so a fenced code example containing a blank line stays in one code block. Blank lines before an outside paragraph or heading still end the list.

Validation:

  • Added 13 focused tests; all pass. Eight of the initial ten failed on the unchanged upstream renderer. Two additional blank-line cases failed before the blank-line handling was added; the third additional test checks outside-block boundaries.
  • The two existing ordinary list-rendering tests pass.
  • TypeScript build, no-emit typecheck, and formatting checks pass.
  • Tests cover both ordered marker delimiters, one-to-four-space marker padding, nested and indented lists, retained code indentation and blank lines, and text following a list.
  • This is a focused correction to the existing renderer, not a claim of full CommonMark conformance. Full repository test suite was not run.

Submitted under the published $0.25 valid bug-fix PR offer: https://agenticjobs.work/jobs/register-your-agent-on-our-new-agentic-job-board . Reward eligibility and payment remain subject to your acceptance. Existing payout coordination can be reused; no new payment address is included here.

@RowletCC

RowletCC commented Sep 20, 2026

Copy link
Copy Markdown
Contributor Author

Payout clarification: native SOL on Solana mainnet is acceptable under the published $0.25-per-valid-bug-fix-PR offer. Please use my existing Solana receiving address: 5wzc9saR1iiPSCLoGS7DU6bbEiCUysUUcPhLx2E7KcRU. A switch to Base USDC is not required. These fixes remain subject to your review and acceptance; please identify the accepted PRs and the net SOL amount in your settlement notice.

@RowletCC RowletCC changed the title Preserve list content indentation across marker widths Preserve code blocks and indentation inside Markdown lists Sep 20, 2026
@ralyodio
ralyodio merged commit 327e807 into profullstack:master Sep 21, 2026
4 checks passed
ralyodio added a commit that referenced this pull request Sep 21, 2026
Each of these PRs passed alone and failed together:

- #149 emits start="N" on ordered lists, so the #161 marker-width tests
  that begin at 12345 now expect the attribute.
- #131 resolves every federated host before connecting, so the #128
  directory salary tests pass allowPrivate like the other federation
  tests do.

Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>
ralyodio added a commit that referenced this pull request Sep 21, 2026
* Use parser decisions to deduplicate resume export headers

* test: reconcile expectations after the 2026-09-21 batch merge

Each of these PRs passed alone and failed together:

- #149 emits start="N" on ordered lists, so the #161 marker-width tests
  that begin at 12345 now expect the attribute.
- #131 resolves every federated host before connecting, so the #128
  directory salary tests pass allowPrivate like the other federation
  tests do.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

---------

Co-authored-by: Anthony Ettinger <anthony@profullstack.com>
Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>
ralyodio added a commit that referenced this pull request Sep 21, 2026
* Continue job import after incomplete structured postings

* test: reconcile expectations after the 2026-09-21 batch merge

Each of these PRs passed alone and failed together:

- #149 emits start="N" on ordered lists, so the #161 marker-width tests
  that begin at 12345 now expect the attribute.
- #131 resolves every federated host before connecting, so the #128
  directory salary tests pass allowPrivate like the other federation
  tests do.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

---------

Co-authored-by: Anthony Ettinger <anthony@profullstack.com>
Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>
ralyodio added a commit that referenced this pull request Sep 21, 2026
* Preserve parentheses in Markdown link destinations

* test: reconcile expectations after the 2026-09-21 batch merge

Each of these PRs passed alone and failed together:

- #149 emits start="N" on ordered lists, so the #161 marker-width tests
  that begin at 12345 now expect the attribute.
- #131 resolves every federated host before connecting, so the #128
  directory salary tests pass allowPrivate like the other federation
  tests do.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

---------

Co-authored-by: Anthony Ettinger <anthony@profullstack.com>
Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>
ralyodio added a commit that referenced this pull request Sep 21, 2026
* Preserve parentheses in Markdown link destinations

* Keep Markdown tables from swallowing following blocks

* test: reconcile expectations after the 2026-09-21 batch merge

Each of these PRs passed alone and failed together:

- #149 emits start="N" on ordered lists, so the #161 marker-width tests
  that begin at 12345 now expect the attribute.
- #131 resolves every federated host before connecting, so the #128
  directory salary tests pass allowPrivate like the other federation
  tests do.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

---------

Co-authored-by: Anthony Ettinger <anthony@profullstack.com>
Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>
ralyodio added a commit that referenced this pull request Sep 21, 2026
* Skip empty Word text elements without consuming following content

* test: reconcile expectations after the 2026-09-21 batch merge

Each of these PRs passed alone and failed together:

- #149 emits start="N" on ordered lists, so the #161 marker-width tests
  that begin at 12345 now expect the attribute.
- #131 resolves every federated host before connecting, so the #128
  directory salary tests pass allowPrivate like the other federation
  tests do.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

---------

Co-authored-by: Anthony Ettinger <anthony@profullstack.com>
Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>
@RowletCC

Copy link
Copy Markdown
Contributor Author

Thank you for merging the fixes. I checked the current repository records: all 33 PRs from this batch are now merged:

#122, #123, #124, #125, #126, #128, #129, #130, #132, #134, #136, #138, #139, #140, #141, #142, #143, #144, #145, #147, #148, #149, #150, #151, #152, #153, #154, #155, #156, #157, #158, #159 and #161.

Under the published $0.25-per-valid-bug-fix-PR offer, that would be $8.25 if all 33 qualify. Could you confirm the eligible count, any deductions, the net SOL amount and expected payment time? Native SOL on Solana mainnet is acceptable, using the receiving address already provided above: 5wzc9saR1iiPSCLoGS7DU6bbEiCUysUUcPhLx2E7KcRU.

I am also available for another bounded batch of ordinary functionality fixes. If you have a priority issue or a preferred scope, please point me to it. This request covers the merged PRs only; it does not add a claim for the profile or promotional post.

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