Fix GraphQL query error: extract item from path instead of querying non-existent field

- Remove 'item' field from GraphQL query in Index.vue (not available in schema)
- Extract item ID from path field (path format: /:item)
- Add paintingItem computed property in ActionBar.vue
- Add getPaintingWithItem() method in Index.vue to add item property
- Update tests to use path instead of item field
- Fix test mocks for g-link and eventBus

Fixes GitHub Action build failure:
Error: Cannot query field "item" on type "Painting"

Build now completes successfully.

Author: hunsalz

src/components/ActionBar.vue

@@ -9,7 +9,7 @@
       :data-favorite="isFavorite"
       role="button"
       aria-label="Toggle favorite"
-      @click.prevent="toggleFavorite(painting.item)"
+      @click.prevent="toggleFavorite(paintingItem)"
     >
       <svg
         xmlns="http://www.w3.org/2000/svg"
@@ -79,12 +79,20 @@ export default {
     };
   },
   computed: {
+    /**
+     * Gets the Wikidata item ID from the painting's path
+     * @returns {string} The Wikidata item ID (e.g., "Q12345")
+     */
+    paintingItem() {
+      // Extract item from path (path format: /:item)
+      return this.painting.path ? this.painting.path.replace(/^\//, "") : null;
+    },
     /**
      * Gets the Wikidata URL for the current painting
      * @returns {string} The full Wikidata URL
      */
     getWikidataLink() {
-      return "https://www.wikidata.org/wiki/" + this.painting.item;
+      return "https://www.wikidata.org/wiki/" + this.paintingItem;
     }
   },
   methods: {

src/components/__tests__/ActionBar.spec.js

@@ -1,9 +1,15 @@
 import { mount } from "@vue/test-utils";
 import ActionBar from "../ActionBar.vue";
+import Vue from "vue";
+
+// Mock Gridsome's g-link component globally
+Vue.component("g-link", { template: "<a><slot /></a>" });
+// Mock the global event bus
+Vue.prototype.$eventBus = new Vue();
 
 describe("ActionBar", () => {
   const mockPainting = {
-    item: "Q12345",
+    path: "/Q12345",
     title: "Test Painting",
     image: "https://example.com/image.jpg",
     cover_image: "https://example.com/cover.jpg"
@@ -18,7 +24,8 @@ describe("ActionBar", () => {
 
     const buttons = wrapper.findAll("button");
     expect(buttons.length).toBe(2); // Favorite and download buttons
-    expect(wrapper.find("g-link").exists()).toBe(true); // Wikidata link
+    // g-link is mocked as <a>, so check for that
+    expect(wrapper.find("a").exists()).toBe(true); // Wikidata link
   });
 
   it("gets correct Wikidata link", () => {

src/pages/Index.vue

@@ -50,7 +50,6 @@ query {
       node {
         id
         path
-        item
         title: paintingLabel
         image
         cover_image: image (width: 770, height: 380, blur: 10)
@@ -230,9 +229,13 @@ export default {
       // ... in case of favorite view
       if (this.view === FAVORITES && this.favorites.length > 0) {
         // filter matching cards
-        return this.$page.paintings.edges.filter(edge =>
-          includes(this.favorites, edge.node.item)
-        );
+        return this.$page.paintings.edges.filter(edge => {
+          // Extract item from path (path format: /:item)
+          const item = edge.node.path
+            ? edge.node.path.replace(/^\//, "")
+            : null;
+          return includes(this.favorites, item);
+        });
       }
       // otherwise show standard dashboard ...
       return this.$page.paintings.edges.filter(edge => {
@@ -248,6 +251,19 @@ export default {
     }
   },
   methods: {
+    /**
+     * Extracts the item ID from a painting's path
+     * @param {Object} painting - The painting node
+     * @returns {Object} Painting object with item property added
+     */
+    getPaintingWithItem(painting) {
+      // Extract item from path (path format: /:item)
+      const item = painting.path ? painting.path.replace(/^\//, "") : null;
+      return {
+        ...painting,
+        item: item
+      };
+    },
     /**
      * Clears all active filters
      */

src/templates/Painting.vue

@@ -113,7 +113,7 @@ query painting ($id: ID!) {
     title: paintingLabel
     image
     year: date (format: "YYYY")
-  	location: locationLabel
+    location: locationLabel
     materials
     depicts
   }

yarn.lock

@@ -1017,10 +1017,10 @@
     minimatch "^3.1.2"
     strip-json-comments "^3.1.1"
 
-"@eslint/js@9.39.1":
-  version "9.39.1"
-  resolved "https://registry.yarnpkg.com/@eslint/js/-/js-9.39.1.tgz#0dd59c3a9f40e3f1882975c321470969243e0164"
-  integrity sha512-S26Stp4zCy88tH94QbBv3XCuzRQiZ9yXofEILmglYTh/Ug/a9/umqvgFtYBAo3Lp0nsI/5/qH1CCrbdK3AP1Tw==
+"@eslint/js@9.39.2":
+  version "9.39.2"
+  resolved "https://registry.yarnpkg.com/@eslint/js/-/js-9.39.2.tgz#2d4b8ec4c3ea13c1b3748e0c97ecd766bdd80599"
+  integrity sha512-q1mjIoW1VX4IvSocvM/vbTiveKC4k9eLrajNEuSsmjymSDEbpGddtpfOoN7YGAqBK3NG+uqo8ia4PDTt8buCYA==
 
 "@eslint/object-schema@^2.1.7":
   version "2.1.7"
@@ -5114,17 +5114,17 @@ eslint-visitor-keys@^4.2.0, eslint-visitor-keys@^4.2.1:
   integrity sha512-Uhdk5sfqcee/9H/rCOJikYz67o0a2Tw2hGRPOG2Y1R2dg7brRe1uG0yaNQDHu+TO/uQPF/5eCapvYSmHUjt7JQ==
 
 eslint@^9.39.1:
-  version "9.39.1"
-  resolved "https://registry.yarnpkg.com/eslint/-/eslint-9.39.1.tgz#be8bf7c6de77dcc4252b5a8dcb31c2efff74a6e5"
-  integrity sha512-BhHmn2yNOFA9H9JmmIVKJmd288g9hrVRDkdoIgRCRuSySRUHH7r/DI6aAXW9T1WwUuY3DFgrcaqB+deURBLR5g==
+  version "9.39.2"
+  resolved "https://registry.yarnpkg.com/eslint/-/eslint-9.39.2.tgz#cb60e6d16ab234c0f8369a3fe7cc87967faf4b6c"
+  integrity sha512-LEyamqS7W5HB3ujJyvi0HQK/dtVINZvd5mAAp9eT5S/ujByGjiZLCzPcHVzuXbpJDJF/cxwHlfceVUDZ2lnSTw==
   dependencies:
     "@eslint-community/eslint-utils" "^4.8.0"
     "@eslint-community/regexpp" "^4.12.1"
     "@eslint/config-array" "^0.21.1"
     "@eslint/config-helpers" "^0.4.2"
     "@eslint/core" "^0.17.0"
     "@eslint/eslintrc" "^3.3.1"
-    "@eslint/js" "9.39.1"
+    "@eslint/js" "9.39.2"
     "@eslint/plugin-kit" "^0.4.1"
     "@humanfs/node" "^0.16.6"
     "@humanwhocodes/module-importer" "^1.0.1"