fix(embed): don't claim an event is online when attendanceMode is unset

rawLocationText() fell through to the "Online" location label whenever
attendanceMode wasn't "online", "in-person", or "hybrid" — i.e. whenever it
was absent entirely. An event that declares no attendance mode at all now
falls back to "Venue not specified" instead, same as an in-person event
with no venue.

Bumps to 0.6.0 (minor) to also release the already-queued playground
restructure (#53) and <ote-subscribe> badge redesign sitting under
Unreleased.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

Author: claude

apps/embed/CHANGELOG.md

@@ -1,6 +1,17 @@
 # @opentechevents/embed changelog
 
-## Unreleased
+## 0.6.0
+
+### Fixed
+
+- An event with no `attendanceMode` declared at all (neither `"online"`,
+  `"in-person"`, nor `"hybrid"`) and no `location` no longer shows an
+  "Online" location label — it now shows "Venue not specified", the same
+  fallback an in-person event with no venue already used. `rawLocationText()`
+  (`apps/embed/src/render.ts`) previously fell through to the "Online"
+  default whenever `attendanceMode` wasn't one of the two handled cases,
+  wrongly asserting online attendance for an event that never declared any
+  attendance mode.
 
 ### Changed
 

apps/embed/index.html

@@ -527,6 +527,15 @@ <h2 id="versioning-heading">Versioning</h2>
                   <a href="https://tools.opentechevents.org/embed/v__EMBED_VERSION__/ote-events.js" target="_blank" rel="noopener">script</a>
                 </td>
               </tr>
+              <tr>
+                <td><code>v0.5.0</code></td>
+                <td><a href="https://opentechevents.org/schema/v0.3/event.schema.json" target="_blank" rel="noopener">v0.3.0</a></td>
+                <td>
+                  <a href="https://github.com/OpenTechEvents/ote-tools/releases/tag/embed-v0.5.0" target="_blank" rel="noopener">release</a>
+                  <span aria-hidden="true"> · </span>
+                  <a href="https://tools.opentechevents.org/embed/v0.5.0/ote-events.js" target="_blank" rel="noopener">script</a>
+                </td>
+              </tr>
               <tr>
                 <td><code>v0.4.0</code></td>
                 <td><a href="https://opentechevents.org/schema/v0.3/event.schema.json" target="_blank" rel="noopener">v0.3.0</a></td>

apps/embed/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@opentechevents/embed",
-  "version": "0.5.0",
+  "version": "0.6.0",
   "oteSpecVersion": "0.3.0",
   "description": "Embeddable <ote-events> web component: drop an OTE feed into any website",
   "license": "MIT",

apps/embed/src/render.ts

@@ -399,15 +399,18 @@ function isOnlineLinkUnpublished(event: PreviewEvent): boolean {
  * `@opentechevents/preview-feed` whenever the source gave no venue and no
  * online URL — a data-agnostic fallback that doesn't know the event's
  * `attendanceMode`. Resolving that sentinel here, where `attendanceMode` is
- * available, avoids two bugs: an in-person event with no venue silently
- * claiming to be "Online" (wrong, not just noisy), and an online event
- * repeating the same "Online" text the attendance badge already shows.
+ * available, avoids three bugs: an in-person event with no venue silently
+ * claiming to be "Online" (wrong, not just noisy); an online event repeating
+ * the same "Online" text the attendance badge already shows; and an event
+ * with no `attendanceMode` at all (nothing declared either way) asserting
+ * "Online" as if that were known — same wrongness as the in-person case,
+ * just with attendance unspecified instead of specified-but-no-venue, so it
+ * shares that case's fallback rather than defaulting to "Online".
  */
 function rawLocationText(event: PreviewEvent, strings: Strings): string {
   if (event.location && event.location !== "online") return event.location;
   if (event.attendanceMode === "online") return strings.onlineLinkUnknown;
-  if (event.attendanceMode === "in-person" || event.attendanceMode === "hybrid") return strings.locationUnknown;
-  return strings.online;
+  return strings.locationUnknown;
 }
 
 function locationText(event: PreviewEvent, strings: Strings): string {

apps/embed/test/element.test.ts

@@ -612,6 +612,25 @@ describe("<ote-events>", () => {
     expect(location?.textContent).not.toContain("Online");
   });
 
+  it("doesn't claim an event with no attendanceMode declared is online", async () => {
+    fetchMock.mockResolvedValue({
+      ok: true,
+      status: 200,
+      text: async () =>
+        JSON.stringify({
+          events: [{ name: "Attendance unspecified", startDate: "2999-01-01" }],
+        }),
+    });
+    const el = createCardsElement();
+    el.setAttribute("feed", "https://example.org/no-attendance-mode.json");
+    document.body.append(el);
+    await flush();
+
+    const location = el.shadowRoot!.querySelector(".event-location");
+    expect(location?.textContent).toBe("Venue not specified");
+    expect(location?.textContent).not.toContain("Online");
+  });
+
   it("defensively hides raw URL locations in cards", () => {
     const container = document.createElement("div");
     renderWidget(container, rawLocationState("cards"));