The handlers for these already exist in core, e.g. RevisionHtmlHandler. But, they're exposed under beta cluster only, see https://en.wikipedia.beta.wmflabs.org/w/rest.php/coredev/v0/revision/1/html because of the 'corded/v0' prefix.

The reason for being under /v0 is that there is this horrific dependency from core to Parsoid extension until T265518 is resolved. All that needs to be done in core is to move the route definitions from coreDevelopmentRoutes.json into coreRoutes.json. But I would really love it if we poked parsing team about moving making parsoid a library before we expose even more Parsoid-dependent API. But if that's not possible, just moving the routes is ok too.

In T284047#7126213, @Pchelolo wrote:

The handlers for these already exist in core, e.g. RevisionHtmlHandler. But, they're exposed under beta cluster only, see https://en.wikipedia.beta.wmflabs.org/w/rest.php/coredev/v0/revision/1/html because of the 'corded/v0' prefix.

The reason for being under /v0 is that there is this horrific dependency from core to Parsoid extension until T265518 is resolved. All that needs to be done in core is to move the route definitions from coreDevelopmentRoutes.json into coreRoutes.json. But I would really love it if we poked parsing team about moving making parsoid a library before we expose even more Parsoid-dependent API. But if that's not possible, just moving the routes is ok too.

I see, thanks for the info on that.

In T284047#7127584, @BPirkle wrote:

From T265518: Move Parsoid ServiceWorker.php and extension/src/Config into core, regarding the parsoid changes:

We'll do this early in 1.37 so we're not in this same position again.

So hopefully we'll be in a position to move the routes cleanly before long. @kostajh , is there a specific time frame regarding the links recommendation project that we should be aware of?

Moving this into "Feature Requests to Review" with Medium priority for now. Let us know if there's an urgent reason that this shouldn't wait for the parsoid changes.

No rush from our end. It can wait for the Parsoid changes. Thanks!

In T284047#7128475, @kostajh wrote:

In T284047#7126213, @Pchelolo wrote:

The handlers for these already exist in core, e.g. RevisionHtmlHandler. But, they're exposed under beta cluster only, see https://en.wikipedia.beta.wmflabs.org/w/rest.php/coredev/v0/revision/1/html because of the 'corded/v0' prefix.

The reason for being under /v0 is that there is this horrific dependency from core to Parsoid extension until T265518 is resolved. All that needs to be done in core is to move the route definitions from coreDevelopmentRoutes.json into coreRoutes.json. But I would really love it if we poked parsing team about moving making parsoid a library before we expose even more Parsoid-dependent API. But if that's not possible, just moving the routes is ok too.

I see, thanks for the info on that.

In T284047#7127584, @BPirkle wrote:

From T265518: Move Parsoid ServiceWorker.php and extension/src/Config into core, regarding the parsoid changes:

We'll do this early in 1.37 so we're not in this same position again.

So hopefully we'll be in a position to move the routes cleanly before long. @kostajh , is there a specific time frame regarding the links recommendation project that we should be aware of?

Moving this into "Feature Requests to Review" with Medium priority for now. Let us know if there's an urgent reason that this shouldn't wait for the parsoid changes.

No rush from our end. It can wait for the Parsoid changes. Thanks!

@BPirkle just checking in on this task, is there an update on timeline?

Does RevisionSourceHandler / /coredev/v0/revision/{id} also depend on Parsoid? The functionality seems unrelated.

Change 752628 had a related patch set uploaded (by Kosta Harlan; author: Kosta Harlan):

[mediawiki/core@master] rest.php: Expose /v1/revision/{id}

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

In T284047#7608490, @Tgr wrote:

Does RevisionSourceHandler / /coredev/v0/revision/{id} also depend on Parsoid? The functionality seems unrelated.

I think the comment about Parsoid may have been specifically for RevisionHtmlHandler.

Change 752630 had a related patch set uploaded (by Kosta Harlan; author: Kosta Harlan):

[mediawiki/core@master] rest: Allow getting page source by revision ID

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

Change 752630 abandoned by Kosta Harlan:

[mediawiki/core@master] rest: Allow getting page source by revision ID

Reason:

Too many other things going on, and we have an alternate implementation in research/mwaddlink repo to get what we need for this

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

Change 752628 abandoned by Kosta Harlan:

[mediawiki/core@master] rest.php: Expose /v1/revision/{id}

Reason:

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

@kostajh is this still relevant or can this ticket be closed out?

In T284047#9643460, @FJoseph-WMF wrote:

@kostajh is this still relevant or can this ticket be closed out?

The work wasn't done, so I think it would make sense to leave it open, even if the team doesn't intend to work on it in.

IMO it's still a useful feature, but it's not something I or (as far as I know) anyone else is waiting on.

https://en.wikipedia.org/w/rest.php/v1/revision/10000/html exists now.