Skip to content

Update TinyPEN controller to use application/ld+json Content-Type#445

Open
Copilot wants to merge 9 commits intodevelopmentfrom
copilot/upgrade-content-type-json-ld
Open

Update TinyPEN controller to use application/ld+json Content-Type#445
Copilot wants to merge 9 commits intodevelopmentfrom
copilot/upgrade-content-type-json-ld

Conversation

Copy link
Contributor

Copilot AI commented Feb 3, 2026

Upgrades all TinyPEN API requests to use application/ld+json content type instead of application/json, aligning with JSON-LD semantics.

Changes

Updated Content-Type header in database/tiny/controller.js:

  • find() - query endpoint
  • save() - create endpoint
  • update() - update endpoint
  • overwrite() - overwrite endpoint
  • remove() - delete endpoint
// Before
headers: {
    'Content-Type': 'application/json; charset=utf-8'
}

// After
headers: {
    'Content-Type': 'application/ld+json; charset=utf-8'
}

Note: Requires corresponding TinyPEN service support (tracked in CenterForDigitalHumanities/TinyPen#9).

Original prompt

This section details on the original issue you should resolve

<issue_title>TinyController application/json+ld Content Type</issue_title>
<issue_description>Upgrade TinyPEN requests to have the content-type "application/json+ld".

At this moment, this is breaking change. This issues is blocked by CenterForDigitalHumanities/TinyPen#9</issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 2 commits January 7, 2026 10:21
* Initial plan

* Fix async/await patterns in Group.js - return promises instead of just awaiting

Co-authored-by: cubap <1119165+cubap@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: cubap <1119165+cubap@users.noreply.github.com>
Co-authored-by: cubap <1119165+cubap@users.noreply.github.com>
Copilot AI changed the title [WIP] Upgrade TinyPEN requests to application/json+ld content type Update TinyPEN controller to use application/ld+json Content-Type Feb 3, 2026
Copilot AI requested a review from cubap February 3, 2026 16:07
cubap and others added 5 commits February 23, 2026 16:08
Introduce an optional shouldUpdate parameter to Group.setMemberRoles and Group.removeMemberRoles to control whether changes are immediately persisted. These methods now skip calling update() when shouldUpdate is false, allowing callers to batch multiple role mutations and perform a single update. Call sites updated: removed a redundant update in Project.removeMember flow and adjusted memberRouter to pass shouldUpdate=false for intermediate role changes, then call group.update() once after all changes. This reduces redundant DB writes and improves efficiency.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@cubap cubap marked this pull request as ready for review February 26, 2026 18:13
@cubap cubap requested a review from thehabes as a code owner February 26, 2026 18:13
@cubap cubap changed the base branch from main to development February 26, 2026 18:13
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.

TinyController application/json+ld Content Type

2 participants