Change canonical reference delimiter from ":" to "." for manuscript formatting

[Copilot]

Manuscripts typically use "." as the section delimiter (e.g., "Sententiae liber 2.dist. 17") rather than ":" which is primarily for biblical references. When building canonical references from structured fields (Document, Section, Subsection), the system was hardcoded to use ":".

Changes

• Updated modifyTableTR() in js/deer-config.js to use "." delimiter when constructing canonical references from structured fields
• Added test coverage for canonical reference construction logic

Behavior

Before:

// Document="Manuscript", Section="2", Subsection="17"
canonicalReference = `${doc} ${section}:${subsection}` // "Manuscript 2:17"

After:

canonicalReference = `${doc} ${section}.${subsection}` // "Manuscript 2.17"

User-entered canonical reference strings (e.g., "Matthew 5:1") are preserved as-is. The structured fields are primarily for filtering/searching and only used to build the reference when no user input exists.

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Default separator for Canonical Reference</issue_title>
<issue_description>The default delimiter for the first segment of a canonical reference locator was set to ":" as in "Matthew 5:4" but most manuscripts would default to just a ".". Allow for the reference locator string to be whatever is entered, but when building it for the interface from {Document, Section, Subsection} be sure to use the "." instead. In most cases, we would rather have the entered string be the displayed value anyway, with the subsection encoding being used mostly for filtering and searching.</issue_description>

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

• Fixes Default separator for Canonical Reference #295

---

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

[cubap]

only reassign canonicalReference if it doesn't already have a value. This whole block can be put inside an if

[cubap]

We don't need this test. The tests here are weak and will be handled separately.