ssastry (Subramanya Sastry)
User

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Monday

  • Clear sailing ahead.

User Details

User Since
Oct 7 2014, 5:34 AM (189 w, 4 d)
Availability
Available
LDAP User
Subramanya Sastry
MediaWiki User
SSastry (WMF)

Recent Activity

Yesterday

ssastry updated the task description for T195263: Do another round of Tidy replacement on May 30th and June 13th (last deploys before final switch).
Fri, May 25, 9:45 PM · User-notice, Tidy, Parsing-Team

Thu, May 24

ssastry closed T194659: Lost attribute (and dirty diff wt2wt) in a table cell as Resolved.
Thu, May 24, 11:48 PM · Patch-For-Review, Parsoid-Rendering, Parsoid
ssastry claimed T195486: html2wt of existing list items, headings, table cells.
Thu, May 24, 11:48 PM · Patch-For-Review, VisualEditor, Parsoid-Serializer, Parsoid
ssastry moved T195414: Heading whitespace normalized outside of modified tree from Backlog to html2wt on the Parsoid board.
Thu, May 24, 11:47 PM · Patch-For-Review, Parsoid
ssastry moved T195486: html2wt of existing list items, headings, table cells from Backlog to html2wt on the Parsoid board.
Thu, May 24, 11:47 PM · Patch-For-Review, VisualEditor, Parsoid-Serializer, Parsoid
ssastry updated the task description for T195263: Do another round of Tidy replacement on May 30th and June 13th (last deploys before final switch).
Thu, May 24, 11:39 PM · User-notice, Tidy, Parsing-Team
ssastry added a comment to T195486: html2wt of existing list items, headings, table cells.

Change 434961 had a related patch set uploaded (by Subramanya Sastry; owner: Subramanya Sastry):
[mediawiki/services/parsoid@master] Minimize whitespace dirty diffs in existing headings, tables, lists

https://gerrit.wikimedia.org/r/434961

Thu, May 24, 5:22 PM · Patch-For-Review, VisualEditor, Parsoid-Serializer, Parsoid
ssastry added a comment to T194806: Edge case parsing bug.

So, we should probably try to fix this.

By "this", I'm going to assume you mean the paragraph wrapping of the abc. The rest of the render looks right to me.

Yes, p-wrapping.

Thu, May 24, 4:56 PM · Parsoid
ssastry added a comment to T194806: Edge case parsing bug.

So, we should probably try to fix this.

By "this", I'm going to assume you mean the paragraph wrapping of the abc. The rest of the render looks right to me.

Thu, May 24, 4:54 PM · Parsoid
ssastry added a project to T195263: Do another round of Tidy replacement on May 30th and June 13th (last deploys before final switch): User-notice.
Thu, May 24, 3:39 PM · User-notice, Tidy, Parsing-Team
ssastry updated the task description for T195263: Do another round of Tidy replacement on May 30th and June 13th (last deploys before final switch).
Thu, May 24, 3:39 PM · User-notice, Tidy, Parsing-Team
ssastry updated the task description for T195263: Do another round of Tidy replacement on May 30th and June 13th (last deploys before final switch).
Thu, May 24, 3:38 PM · User-notice, Tidy, Parsing-Team
ssastry added a comment to T195263: Do another round of Tidy replacement on May 30th and June 13th (last deploys before final switch).

Given that there are by definition no reader-facing pages on Wikidata that have wikitext on them (all the reader-facing pages are structured data pages), converting it over to Remex seems fine. :-)

Thu, May 24, 3:38 PM · User-notice, Tidy, Parsing-Team
ssastry added a comment to T195414: Heading whitespace normalized outside of modified tree.

Btw, just want to draw attention to this comment,

16:43 <+arlolra> why is it that new heading are added with the spacing
16:43 <+arlolra> https://en.wikipedia.org/w/index.php?title=Blinn_College&curid=1246005&diff=842658946&oldid=842657772&diffmode=source
16:44 <+arlolra> it doesn't seem like we should be adding that if we're not going to preserve it

No, these two things are somewhat orthogonal. (a) Whitespace trimming in wikitext -> html is wikitext spec (b) Editors / wikis want whitespace in lists, headings, tables, etc. for readability and so, new content should continue to add them.

The question however is what do we do about headings /list items / table cells that were already present in original wikitext. That is where this gets tricky. There is no good automatic solution. I'm going to file a separate ticket for it.

Thu, May 24, 1:50 PM · Patch-For-Review, Parsoid
ssastry added a project to T195486: html2wt of existing list items, headings, table cells: VisualEditor.
Thu, May 24, 1:49 PM · Patch-For-Review, VisualEditor, Parsoid-Serializer, Parsoid
ssastry triaged T195486: html2wt of existing list items, headings, table cells as High priority.
Thu, May 24, 1:49 PM · Patch-For-Review, VisualEditor, Parsoid-Serializer, Parsoid
ssastry updated the task description for T195486: html2wt of existing list items, headings, table cells.
Thu, May 24, 1:49 PM · Patch-For-Review, VisualEditor, Parsoid-Serializer, Parsoid
ssastry created T195486: html2wt of existing list items, headings, table cells.
Thu, May 24, 1:48 PM · Patch-For-Review, VisualEditor, Parsoid-Serializer, Parsoid
ssastry added a comment to T195414: Heading whitespace normalized outside of modified tree.

Btw, just want to draw attention to this comment,

16:43 <+arlolra> why is it that new heading are added with the spacing
16:43 <+arlolra> https://en.wikipedia.org/w/index.php?title=Blinn_College&curid=1246005&diff=842658946&oldid=842657772&diffmode=source
16:44 <+arlolra> it doesn't seem like we should be adding that if we're not going to preserve it

Thu, May 24, 1:35 PM · Patch-For-Review, Parsoid
ssastry added a comment to T195414: Heading whitespace normalized outside of modified tree.

This also explains the mysterious html2wt perf degradation since mid-December that I noticed recently.

https://grafana.wikimedia.org/dashboard/db/parsoid-timing-html2wt?panelId=26&fullscreen&orgId=1&from=now-1y&to=now

There is a noticeable upward bump around mid-December ... and guess what ... html5 ids with fallbacks was deployed on December 12.

Thu, May 24, 1:30 PM · Patch-For-Review, Parsoid

Wed, May 23

ssastry added a comment to T195414: Heading whitespace normalized outside of modified tree.

This also explains the mysterious html2wt perf degradation since mid-December that I noticed recently.

Wed, May 23, 11:12 PM · Patch-For-Review, Parsoid
ssastry added a comment to T195414: Heading whitespace normalized outside of modified tree.

As expected, it is the fallback ids (!).

Wed, May 23, 11:06 PM · Patch-For-Review, Parsoid

Tue, May 22

ssastry added a comment to T195317: Comment prevents category from being migrated out of list.

All these test cases are fixed with that patch there.

Tue, May 22, 9:18 PM · Patch-For-Review, Parsoid
ssastry moved T195330: Nowiki protection of sol whitespace in wikilink titles probably unnecessary from Backlog to html2wt on the Parsoid board.
Tue, May 22, 8:46 PM · Parsoid-Serializer, Parsoid
ssastry added a comment to T195174: Edge case tokenizing difference.

Yes, I think so.

Tue, May 22, 8:41 PM · Patch-For-Review, Parsoid-Tokenizer, Parsoid
ssastry triaged T194945: Infobox overlaps with text, prevents editing the latter. as Normal priority.
Tue, May 22, 2:56 PM · Parsoid, VisualEditor
ssastry moved T195227: Template serialization can dirty diff transclusions with 'constructor' as a template arg from Backlog to html2wt on the Parsoid board.
Tue, May 22, 2:51 PM · Patch-For-Review, Parsoid-Serializer, Parsoid
ssastry moved T195272: Maximum call stack size exceeded from Backlog to Performance on the Parsoid board.
Tue, May 22, 2:51 PM · Parsoid
ssastry closed T194658: Empty table cells aren't serialized correctly as Resolved.
Tue, May 22, 2:46 PM · Patch-For-Review, Parsoid-Serializer, Parsoid-Nowiki, Parsoid
ssastry closed T194763: stripUnnecessaryIndentPreNowikis in wikitext serializer is over-aggressive and can introduce semantic diffs as Resolved.
Tue, May 22, 2:46 PM · Patch-For-Review, Parsoid-Serializer, Parsoid
ssastry closed T194777: Category links are swallowed into definition list output even when they are outside the list as Resolved.
Tue, May 22, 2:46 PM · Patch-For-Review, Parsoid

Mon, May 21

ssastry updated the task description for T195263: Do another round of Tidy replacement on May 30th and June 13th (last deploys before final switch).
Mon, May 21, 9:29 PM · User-notice, Tidy, Parsing-Team
ssastry triaged T195263: Do another round of Tidy replacement on May 30th and June 13th (last deploys before final switch) as Normal priority.
Mon, May 21, 8:37 PM · User-notice, Tidy, Parsing-Team
ssastry triaged T195227: Template serialization can dirty diff transclusions with 'constructor' as a template arg as Normal priority.
Mon, May 21, 2:48 AM · Patch-For-Review, Parsoid-Serializer, Parsoid
ssastry created T195227: Template serialization can dirty diff transclusions with 'constructor' as a template arg.
Mon, May 21, 2:48 AM · Patch-For-Review, Parsoid-Serializer, Parsoid

Sun, May 20

ssastry moved T195171: Bad parse (possibly because of broken argument source offsets) from Backlog to Read Views on the Parsoid board.
Sun, May 20, 4:47 AM · Patch-For-Review, Parsoid
ssastry moved T195174: Edge case tokenizing difference from Backlog to Read Views on the Parsoid board.
Sun, May 20, 4:46 AM · Patch-For-Review, Parsoid-Tokenizer, Parsoid
ssastry triaged T195174: Edge case tokenizing difference as Normal priority.
Sun, May 20, 4:32 AM · Patch-For-Review, Parsoid-Tokenizer, Parsoid
ssastry created T195174: Edge case tokenizing difference.
Sun, May 20, 4:32 AM · Patch-For-Review, Parsoid-Tokenizer, Parsoid
ssastry triaged T195171: Bad parse (possibly because of broken argument source offsets) as Normal priority.
Sun, May 20, 2:34 AM · Patch-For-Review, Parsoid
ssastry created T195171: Bad parse (possibly because of broken argument source offsets).
Sun, May 20, 2:34 AM · Patch-For-Review, Parsoid

Fri, May 18

ssastry added a comment to T187951: Intermittent "Error loading data from server" error using VE in Firefox and Chrome on officewiki.

So, this is a bug introduced by database changes (or a bug elsewhere exposed by those database changes). Looking at https://www.mediawiki.org/wiki/MediaWiki_1.32/wmf.1#Core_changes, there are 6 RDBMS related entries there. So, @aaron or @Krinkle should be able to take it from here.

Fri, May 18, 8:08 PM · Performance-Team, VisualEditor (Current work)
ssastry edited projects for T187951: Intermittent "Error loading data from server" error using VE in Firefox and Chrome on officewiki, added: Performance-Team; removed Parsoid.
Fri, May 18, 7:59 PM · Performance-Team, VisualEditor (Current work)
ssastry updated subscribers of T187951: Intermittent "Error loading data from server" error using VE in Firefox and Chrome on officewiki.

See https://logstash.wikimedia.org/goto/75432740fdaa6f43376ce203b0956d07 -- that is a search for officewiki events that are not INFO events for the April 1 - May 18 time period. You can see that the event graph starts around April 18th.

Fri, May 18, 7:59 PM · Performance-Team, VisualEditor (Current work)
ssastry added a comment to T187951: Intermittent "Error loading data from server" error using VE in Firefox and Chrome on officewiki.

It's definitely been a problem for more than 6 months. This bug was filed
in Feb, but Sherry and I were experiencing it for longer.

Fri, May 18, 2:24 PM · Performance-Team, VisualEditor (Current work)
ssastry added a comment to T187951: Intermittent "Error loading data from server" error using VE in Firefox and Chrome on officewiki.

@Deskana, @Esanders maybe you guys can see if perf / platform team folks might have any insight into this at the hackathon. I'll separately poke at instrumenting / tracing parsoid monday.

Fri, May 18, 7:13 AM · Performance-Team, VisualEditor (Current work)
ssastry added a comment to T187951: Intermittent "Error loading data from server" error using VE in Firefox and Chrome on officewiki.

See timings from about April 19th to May 2nd. You will see the shift happen around Apr 25.

Fri, May 18, 7:09 AM · Performance-Team, VisualEditor (Current work)
ssastry added a comment to T187951: Intermittent "Error loading data from server" error using VE in Firefox and Chrome on officewiki.

Ok, so I got curious and grepped older logs .. and it seems like this behavior started the week of Apr 24.

Fri, May 18, 6:55 AM · Performance-Team, VisualEditor (Current work)
ssastry added a comment to T187951: Intermittent "Error loading data from server" error using VE in Firefox and Chrome on officewiki.

So, here is what I found so far over the last hour:

  1. When I copied over my officewiki cookie to wtp1025 and use that cookie to run a local commandline parse of officewiki pages, they are consistently parse very quickly
  2. When I restrict myself to a single small page (say User:Ssastry), and if I don't actually make an edit, the page opens in VE quickly almost very consistently
  3. When I restrict myself to a single small page (say User:Ssastry), and I click Edit Source (and not do anything else), click Read/History, and open the page in VE, it is once again slow (for 2 attempts), and back to being fast again.
  4. There are also parse time interactions when multiple pages are open in different tabs.
Fri, May 18, 6:14 AM · Performance-Team, VisualEditor (Current work)

Thu, May 17

ssastry added a comment to T187951: Intermittent "Error loading data from server" error using VE in Firefox and Chrome on officewiki.

So, when I hacked my local parsoid to hardcode my cookies into the code and parse the officewiki pages, they all complete in 2s or less. So, something specific to the production setup. Will investigate tomorrow.

Thu, May 17, 9:47 PM · Performance-Team, VisualEditor (Current work)
ssastry added a comment to T187951: Intermittent "Error loading data from server" error using VE in Firefox and Chrome on officewiki.

Deskana: all the pages (included the SEO log) page loaded instantaneously, but I think that is because your loads had already cached it.

I'm having this problem in Safari. I think that the second edit on a page (in the same session) may be the problem.

I think there is something to this here. To test this, I edited https://office.wikimedia.org/wiki/User:Ssastry and the next load took 20+ seconds ( I could repro. this 2 times) .. the 20+ second is suspiciously long and looks like something timing out before the parse succeeds.

Thu, May 17, 9:05 PM · Performance-Team, VisualEditor (Current work)
ssastry added a comment to T187951: Intermittent "Error loading data from server" error using VE in Firefox and Chrome on officewiki.

Deskana: all the pages (included the SEO log) page loaded instantaneously, but I think that is because your loads had already cached it.

Thu, May 17, 9:04 PM · Performance-Team, VisualEditor (Current work)
ssastry added a comment to T187951: Intermittent "Error loading data from server" error using VE in Firefox and Chrome on officewiki.

Somehow, I finally experienced this problem.

The problem is this query (or its equivalent for whatever page you're on): https://office.wikimedia.org/w/api.php?action=visualeditor&format=json&paction=parse&page=User%3ADeskana_(WMF)%2FSEO_project_activity_log&uselang=en

Pasting the URL into my browser, I can see that sometimes it takes excessively long for it to resolve.

Thu, May 17, 8:44 PM · Performance-Team, VisualEditor (Current work)
ssastry closed T194821: parsoid warning with WikibaseLexeme on test.wikidata & beta "Unknown contentmodel wikibase-lexeme" as Resolved.
Thu, May 17, 5:27 PM · Patch-For-Review, Beta-Cluster-reproducible, Wikidata, Lexicographical data, Wikimedia-log-errors, Parsoid
ssastry closed T194821: parsoid warning with WikibaseLexeme on test.wikidata & beta "Unknown contentmodel wikibase-lexeme", a subtask of T191457: Deploy WikibaseLexeme on www.wikidata.org, as Resolved.
Thu, May 17, 5:27 PM · User-Addshore, Patch-For-Review, Wikidata-Turtles-Sprint #5, Lexicographical data, Wikidata
ssastry placed T193482: false positive report on Linter stripped-tag results up for grabs.
Thu, May 17, 4:17 PM · MediaWiki-extensions-Linter, Parsoid
ssastry closed T193482: false positive report on Linter stripped-tag results as Invalid.

See https://de.wikipedia.org/w/api.php?action=parse&page=Vorlage%3ATabMenu%2FDoku%2FTMT%2F1R&format=json to see a stray </div> in there. It comes from {{TabMenu | Mode=EndTabMenu }}

Thu, May 17, 4:12 PM · MediaWiki-extensions-Linter, Parsoid
ssastry triaged T194565: Parsoid escaping of internal link with query string different to PHP parser as Normal priority.
Thu, May 17, 3:56 PM · Parsoid, VisualEditor
ssastry triaged T193500: Parsoid shouldn't strip sections tags if editing client hasn't as Normal priority.
Thu, May 17, 3:56 PM · Parsoid
ssastry moved T194565: Parsoid escaping of internal link with query string different to PHP parser from Backlog to Links on the Parsoid board.
Thu, May 17, 3:56 PM · Parsoid, VisualEditor
ssastry claimed T194821: parsoid warning with WikibaseLexeme on test.wikidata & beta "Unknown contentmodel wikibase-lexeme".
Thu, May 17, 3:55 PM · Patch-For-Review, Beta-Cluster-reproducible, Wikidata, Lexicographical data, Wikimedia-log-errors, Parsoid

Wed, May 16

ssastry claimed T194658: Empty table cells aren't serialized correctly.
Wed, May 16, 7:50 PM · Patch-For-Review, Parsoid-Serializer, Parsoid-Nowiki, Parsoid
ssastry claimed T194777: Category links are swallowed into definition list output even when they are outside the list.
Wed, May 16, 7:48 PM · Patch-For-Review, Parsoid
ssastry closed T193685: Enable RemexHTML on all wikis with < 100 linter errors in all high priority linter categories in ns0 (main namespace) as Resolved.

I will update the queries to remove those that have Tidy removed -- so they can continue to be used.

Wed, May 16, 5:42 PM · Patch-For-Review, Tech-Ambassadors, User-notice, Parsing-Team, RemexHtml, Tidy
ssastry closed T193685: Enable RemexHTML on all wikis with < 100 linter errors in all high priority linter categories in ns0 (main namespace), a subtask of T175706: Progressively switch Wikimedia wikis from Tidy to RemexHTML, as Resolved.
Wed, May 16, 5:42 PM · Wikimedia-Site-requests, Tidy, Parsing-Team
ssastry updated the task description for T175706: Progressively switch Wikimedia wikis from Tidy to RemexHTML.
Wed, May 16, 5:41 PM · Wikimedia-Site-requests, Tidy, Parsing-Team
ssastry moved T194777: Category links are swallowed into definition list output even when they are outside the list from Backlog to Read Views on the Parsoid board.
Wed, May 16, 1:40 PM · Patch-For-Review, Parsoid
ssastry moved T194806: Edge case parsing bug from Backlog to Read Views on the Parsoid board.
Wed, May 16, 3:17 AM · Parsoid
ssastry triaged T194806: Edge case parsing bug as Normal priority.
Wed, May 16, 3:17 AM · Parsoid
ssastry created T194806: Edge case parsing bug.
Wed, May 16, 3:17 AM · Parsoid
ssastry updated the task description for T193685: Enable RemexHTML on all wikis with < 100 linter errors in all high priority linter categories in ns0 (main namespace).
Wed, May 16, 1:21 AM · Patch-For-Review, Tech-Ambassadors, User-notice, Parsing-Team, RemexHtml, Tidy

Tue, May 15

ssastry triaged T194777: Category links are swallowed into definition list output even when they are outside the list as Normal priority.
Tue, May 15, 4:13 PM · Patch-For-Review, Parsoid
ssastry created T194777: Category links are swallowed into definition list output even when they are outside the list.
Tue, May 15, 4:13 PM · Patch-For-Review, Parsoid
ssastry added a comment to T194658: Empty table cells aren't serialized correctly.

I decided to eliminate the nowikis and instead a whitespace char in all empty cells which is a much better approach.

Tue, May 15, 2:13 PM · Patch-For-Review, Parsoid-Serializer, Parsoid-Nowiki, Parsoid
ssastry renamed T194658: Empty table cells aren't serialized correctly from Missing required nowiki in table syntax during html2wt to Empty table cells aren't serialized correctly.
Tue, May 15, 2:13 PM · Patch-For-Review, Parsoid-Serializer, Parsoid-Nowiki, Parsoid
ssastry moved T194763: stripUnnecessaryIndentPreNowikis in wikitext serializer is over-aggressive and can introduce semantic diffs from Backlog to html2wt on the Parsoid board.
Tue, May 15, 2:12 PM · Patch-For-Review, Parsoid-Serializer, Parsoid
ssastry triaged T194763: stripUnnecessaryIndentPreNowikis in wikitext serializer is over-aggressive and can introduce semantic diffs as Normal priority.
Tue, May 15, 2:11 PM · Patch-For-Review, Parsoid-Serializer, Parsoid
ssastry created T194763: stripUnnecessaryIndentPreNowikis in wikitext serializer is over-aggressive and can introduce semantic diffs.
Tue, May 15, 2:11 PM · Patch-For-Review, Parsoid-Serializer, Parsoid

Mon, May 14

ssastry updated the task description for T194658: Empty table cells aren't serialized correctly.
Mon, May 14, 10:35 PM · Patch-For-Review, Parsoid-Serializer, Parsoid-Nowiki, Parsoid
ssastry added a comment to T194658: Empty table cells aren't serialized correctly.

Oh hmm .. bad test case.

Mon, May 14, 10:34 PM · Patch-For-Review, Parsoid-Serializer, Parsoid-Nowiki, Parsoid
ssastry updated the task description for T194658: Empty table cells aren't serialized correctly.
Mon, May 14, 10:33 PM · Patch-For-Review, Parsoid-Serializer, Parsoid-Nowiki, Parsoid
ssastry added a comment to T194658: Empty table cells aren't serialized correctly.

Oh hmm .. bad test case.

[subbu@earth:~/work/wmf/mediawiki] echo "<table><tr><td>x</td><td data-parsoid='{&quot;stx&quot;:&quot;row&quot;}'></td><td data-parsoid='{&quot;stx&quot;:&quot;row&quot;}'>y</td></tr></table>" | parse.js --html2wt | parse.js --normalize=parsoid
Mon, May 14, 10:27 PM · Patch-For-Review, Parsoid-Serializer, Parsoid-Nowiki, Parsoid
ssastry renamed T194658: Empty table cells aren't serialized correctly from Misisng required nowiki in table syntax during html2wt to Missing required nowiki in table syntax during html2wt.
Mon, May 14, 9:54 PM · Patch-For-Review, Parsoid-Serializer, Parsoid-Nowiki, Parsoid
ssastry added a comment to T192037: Writeup some sort of position statement against subsets of wikitext.

But, if / where subsets are absolutely required, they should be handled systematically and uniformly everywhere (extensions, templates, messages, UI, etc.).

Mon, May 14, 6:00 PM · Parsing-Team
ssastry added a comment to T192037: Writeup some sort of position statement against subsets of wikitext.

Related: T191939

Mon, May 14, 5:59 PM · Parsing-Team
ssastry updated the task description for T193685: Enable RemexHTML on all wikis with < 100 linter errors in all high priority linter categories in ns0 (main namespace).
Mon, May 14, 5:31 PM · Patch-For-Review, Tech-Ambassadors, User-notice, Parsing-Team, RemexHtml, Tidy
ssastry added a comment to T157418: RFC: Make some aspects of Tidy's whitespace stripping behavior part of wikitext parsing "spec".

Our round trip testing infrastructure compares roundtripped wikitext with original wikitext. In order to figure out which diffs are syntactic and which are semantic, it does diffing with a bunch of normalizations and other tricks. But, those tricks are proving insufficient with the kind of normalization that is happening with this whitespace trimming. So, we need to update our round-trip diffing strategy. In order to not block other Parsoid deployments (since all deployments have to go through round trip testing and not have any unexplainable semantic diffs), I am reverting this patch till then.

Is there a task for that separate work (add as blocker to this task)?

Mon, May 14, 5:01 PM · TechCom-RFC (TechCom-Approved), Patch-For-Review, Tidy, Community-Liaisons, Parsoid, MediaWiki-Parser
ssastry reopened T157418: RFC: Make some aspects of Tidy's whitespace stripping behavior part of wikitext parsing "spec" as "Open".
Mon, May 14, 4:19 PM · TechCom-RFC (TechCom-Approved), Patch-For-Review, Tidy, Community-Liaisons, Parsoid, MediaWiki-Parser
ssastry added a comment to T157418: RFC: Make some aspects of Tidy's whitespace stripping behavior part of wikitext parsing "spec".

Change 433004 had a related patch set uploaded (by Subramanya Sastry; owner: Subramanya Sastry):
[mediawiki/services/parsoid@master] Revert "RFC T157418: Trim whitespace in wikitext headings, lists, tables"

https://gerrit.wikimedia.org/r/433004

Mon, May 14, 4:18 PM · TechCom-RFC (TechCom-Approved), Patch-For-Review, Tidy, Community-Liaisons, Parsoid, MediaWiki-Parser
ssastry moved T194658: Empty table cells aren't serialized correctly from Backlog to html2wt on the Parsoid board.
Mon, May 14, 2:23 PM · Patch-For-Review, Parsoid-Serializer, Parsoid-Nowiki, Parsoid
ssastry moved T194659: Lost attribute (and dirty diff wt2wt) in a table cell from Backlog to Read Views on the Parsoid board.
Mon, May 14, 2:23 PM · Patch-For-Review, Parsoid-Rendering, Parsoid
ssastry triaged T194659: Lost attribute (and dirty diff wt2wt) in a table cell as Normal priority.
Mon, May 14, 2:23 PM · Patch-For-Review, Parsoid-Rendering, Parsoid
ssastry created T194659: Lost attribute (and dirty diff wt2wt) in a table cell.
Mon, May 14, 2:23 PM · Patch-For-Review, Parsoid-Rendering, Parsoid
ssastry triaged T194658: Empty table cells aren't serialized correctly as Normal priority.
Mon, May 14, 2:20 PM · Patch-For-Review, Parsoid-Serializer, Parsoid-Nowiki, Parsoid
ssastry created T194658: Empty table cells aren't serialized correctly.
Mon, May 14, 2:20 PM · Patch-For-Review, Parsoid-Serializer, Parsoid-Nowiki, Parsoid

Sat, May 12

ssastry updated the task description for T193685: Enable RemexHTML on all wikis with < 100 linter errors in all high priority linter categories in ns0 (main namespace).
Sat, May 12, 10:54 PM · Patch-For-Review, Tech-Ambassadors, User-notice, Parsing-Team, RemexHtml, Tidy

Fri, May 11

ssastry added a comment to T193685: Enable RemexHTML on all wikis with < 100 linter errors in all high priority linter categories in ns0 (main namespace).

I've been checking linter errors on eswiktionary, it should qualify now.

Fri, May 11, 6:31 PM · Patch-For-Review, Tech-Ambassadors, User-notice, Parsing-Team, RemexHtml, Tidy
ssastry updated the task description for T193685: Enable RemexHTML on all wikis with < 100 linter errors in all high priority linter categories in ns0 (main namespace).
Fri, May 11, 5:45 PM · Patch-For-Review, Tech-Ambassadors, User-notice, Parsing-Team, RemexHtml, Tidy
ssastry added a comment to T194082: Cannot read property 'insertBefore' of null.

This crasher looks like the result of https://github.com/wikimedia/parsoid/commit/58d49a8588b7831af0e3385315f01c72a60b5936

When the sentinel itself is normalized away (it being a text node and as a result of the call to a.normalize()) it is left dangling and eventually throws while trying to addDiffMarks around it.

Fri, May 11, 3:00 AM · Patch-For-Review, Parsoid

Thu, May 10

ssastry updated the task description for T193685: Enable RemexHTML on all wikis with < 100 linter errors in all high priority linter categories in ns0 (main namespace).
Thu, May 10, 2:34 PM · Patch-For-Review, Tech-Ambassadors, User-notice, Parsing-Team, RemexHtml, Tidy