@smarterplan/ngx-smarterplan-core 1.4.13 → 1.4.15

package/fesm2022/smarterplan-ngx-smarterplan-core.mjs

@@ -1207,6 +1207,22 @@ class MissionService {
     async getMissionsForSpace(spaceID) {
         return this.API.__proto__.MissionsBySpace(spaceID).then((missions) => missions.items);
     }
+    /**
+     * Same as getMissionsForSpace(), but follows `nextToken` until the connection is
+     * exhausted instead of returning only the first page.
+     */
+    async getAllMissionsForSpace(spaceID) {
+        const all = [];
+        let nextToken = null;
+        do {
+            const page = await this.API.__proto__.MissionsBySpace(spaceID, undefined, undefined, 1000, nextToken);
+            if (page?.items?.length > 0) {
+                all.push(...page.items.filter((mis) => !!mis && !mis.deletedAt));
+            }
+            nextToken = page?.nextToken ?? null;
+        } while (nextToken);
+        return all;
+    }
     activateMission(mis) {
         if (mis &&
             (!mis.groups || !mis.groups.includes(ProfileStatus.ACTIVE))) {
@@ -6000,6 +6016,34 @@ const copyFileInS3 = async (sourceKey, destKey) => {
     const result = await Storage.copy({ key: sourceKey, level: "public" }, { key: destKey, level: "public" });
     return result.key;
 };
+/**
+ * Fallback duplication: downloads the object and re-uploads it under the new key.
+ *
+ * Storage.copy() builds the S3 `x-amz-copy-source` header by plain string concatenation
+ * (see @aws-amplify/storage AWSS3Provider.copy: `${bucket}/${srcPrefix}${srcKey}`), with no
+ * URL-encoding of the key. A key holding a space, an accent or a parenthesis therefore
+ * yields a header that does not match the signed canonical request, and S3 answers
+ * "The request signature we calculated does not match the signature you provided".
+ *
+ * Round-tripping the bytes avoids that header entirely and works for any key, at the cost
+ * of transferring the file through the browser — hence its use as a fallback only.
+ * IMPORTANT: like copyFileInS3, this must NOT swallow errors.
+ */
+const copyFileInS3ViaDownload = async (sourceKey, destKey) => {
+    const downloaded = await Storage.get(sourceKey, {
+        download: true,
+        level: "public",
+    });
+    const body = downloaded?.Body;
+    if (!body) {
+        throw new Error(`Empty download for "${sourceKey}", cannot duplicate it`);
+    }
+    const result = await Storage.put(destKey, body, {
+        level: "public",
+        contentType: downloaded.ContentType || undefined,
+    });
+    return result.key;
+};
 
 function getDistanceBetweenTwoPoints(startPoint, endPoint) {
     const dx = endPoint.x - startPoint.x;
@@ -8771,7 +8815,9 @@ class EquipmentService extends BaseObjectService {
                     const subEquips = [];
                     await Promise.all(equip.subequipmentsIDs.map(async (subId) => {
                         const subEquip = await this.API.GetEquipment(subId);
-                        subEquips.push(subEquip);
+                        if (subEquip) {
+                            subEquips.push(subEquip);
+                        }
                     }));
                     equip.subEquipments = subEquips;
                 }
@@ -8908,12 +8954,14 @@ class EquipmentService extends BaseObjectService {
      * S3 object itself has already been physically copied by the caller beforehand,
      * so this only needs to record the Note pointing at that new key.
      */
-    async createNoteWithExistingAttachment(equipmentID, attachmentKey, type, ownerMissionID) {
+    async createNoteWithExistingAttachment(equipmentID, attachmentKey, type, ownerMissionID, dates = {}) {
         return this.API.__proto__.CreateNote({
             attachment: attachmentKey,
             type,
             equipmentID,
             ownerMissionID,
+            signDate: dates.signDate ?? null,
+            expirationDate: dates.expirationDate ?? null,
         });
     }
     async uploadImage(equipment, imageFile) {
@@ -9627,6 +9675,33 @@ class HashtagService {
     async getHashtag(hashtagID) {
         return this.API.__proto__.GetHashtag(hashtagID);
     }
+    /*
+     * ---------------------------------------------------------------------------
+     * Pure data access — NO side effect on currentHashtags / currentSpaceID.
+     *
+     * `currentHashtags` is shared UI state read by HashtagFromIdPipe to render a
+     * keyword's name. Batch work (space duplication, export, admin tooling) touches
+     * hashtags of spaces the operator is not editing, and must not reshape what the
+     * screens are displaying — hence this side-effect-free trio.
+     * ---------------------------------------------------------------------------
+     */
+    /** Reads a space's keywords without touching the shared cache. */
+    async listHashtagsForSpace(spaceID) {
+        return this.API.__proto__.HashtagsBySpace(spaceID).then((response) => response.items);
+    }
+    /** Creates a keyword in a space without adding it to the shared cache. */
+    async createHashtagForSpace(name, spaceID) {
+        return this.API.__proto__.CreateHashtag({ name, spaceID });
+    }
+    /** Deletes a keyword without touching the shared cache. */
+    async deleteHashtagById(hashtagID) {
+        return this.API.__proto__.DeleteHashtag({ id: hashtagID });
+    }
+    /*
+     * ---------------------------------------------------------------------------
+     * UI-facing operations — same actions, plus maintenance of the shared cache.
+     * ---------------------------------------------------------------------------
+     */
     async createHashtag(hashtag) {
         const hashtagCreated = await this.API.__proto__.CreateHashtag(hashtag);
         this.currentHashtags.push(hashtagCreated);
@@ -9640,7 +9715,7 @@ class HashtagService {
             return this.currentHashtags;
         }
         this.currentSpaceID = spaceID;
-        return this.API.__proto__.HashtagsBySpace(spaceID).then((response) => response.items);
+        return this.listHashtagsForSpace(spaceID);
     }
     static ɵfac = i0.ɵɵngDeclareFactory({ minVersion: "12.0.0", version: "17.3.12", ngImport: i0, type: HashtagService, deps: [{ token: 'currentAPIService' }], target: i0.ɵɵFactoryTarget.Injectable });
     static ɵprov = i0.ɵɵngDeclareInjectable({ minVersion: "12.0.0", version: "17.3.12", ngImport: i0, type: HashtagService, providedIn: "root" });
@@ -9660,14 +9735,33 @@ class HashtagFromIdPipe {
     constructor(hashtagService) {
         this.hashtagService = hashtagService;
     }
+    /**
+     * Resolves a hashtag id to its display name.
+     *
+     * The cache is consulted for THIS id, and the network answers whenever the id is not
+     * in it.
+     *
+     * Hashtag ids are globally unique, so a cache holding entries from several spaces can
+     * only ever produce a correct hit or a miss, never a wrong name.
+     */
     async transform(hashtagID) {
-        if (this.hashtagService.currentHashtags.length === 0) {
-            return this.hashtagService.getHashtag(hashtagID).then((hashtag) => {
-                return hashtag.name;
-            });
+        if (!hashtagID) {
+            return "";
+        }
+        const cached = this.hashtagService.currentHashtags.find((_hashtag) => _hashtag.id === hashtagID);
+        if (cached) {
+            return cached.name ?? "";
+        }
+        try {
+            const hashtag = await this.hashtagService.getHashtag(hashtagID);
+            return hashtag?.name ?? "";
+        }
+        catch (error) {
+            // A keyword that cannot be read (deleted, network hiccup) must not break the
+            // rendering of the POI that carries it.
+            console.warn(`[HashtagFromIdPipe] Could not resolve the keyword "${hashtagID}":`, error);
+            return "";
         }
-        const hashtag = this.hashtagService.currentHashtags.find((_hashtag) => _hashtag.id === hashtagID);
-        return hashtag ? hashtag.name : "";
     }
     static ɵfac = i0.ɵɵngDeclareFactory({ minVersion: "12.0.0", version: "17.3.12", ngImport: i0, type: HashtagFromIdPipe, deps: [{ token: HashtagService }], target: i0.ɵɵFactoryTarget.Pipe });
     static ɵpipe = i0.ɵɵngDeclarePipe({ minVersion: "14.0.0", version: "17.3.12", ngImport: i0, type: HashtagFromIdPipe, name: "hashtagFromID" });
@@ -10387,6 +10481,40 @@ class CommentService {
     async createComment(comment) {
         return this.API.__proto__.CreateComment(comment);
     }
+    /**
+     * Returns EVERY comment attached to a feature, following `nextToken` until the
+     * connection is exhausted.
+     *
+     * The `comments` connection nested inside FeaturesBySpace/GetFeature is capped by
+     * the AppSync resolver (`$util.defaultIfNull($context.args.limit, 100)`) and exposes
+     * a `nextToken` that nobody follows, so a media-rich POI silently loses every
+     * attachment past the first page. Anything that must see the complete media set of
+     * a feature (duplication, export, audit) has to use this method rather than
+     * `feature.comments.items`.
+     */
+    async getAllCommentsForFeature(featureID) {
+        return this.getAllCommentsPaginated((limit, nextToken) => this.API.__proto__.CommentsByFeature(featureID, undefined, undefined, limit, nextToken), featureID);
+    }
+    /** Same as getAllCommentsForFeature(), for the attachments of a ticket. */
+    async getAllCommentsForTicket(ticketID) {
+        return this.getAllCommentsPaginated((limit, nextToken) => this.API.__proto__.CommentsByTicket(ticketID, undefined, undefined, limit, nextToken), ticketID);
+    }
+    /** Drains a paginated comment query until its nextToken is exhausted. */
+    async getAllCommentsPaginated(fetchPage, ownerID) {
+        if (!ownerID) {
+            return [];
+        }
+        const all = [];
+        let nextToken = null;
+        do {
+            const page = await fetchPage(1000, nextToken);
+            if (page?.items?.length > 0) {
+                all.push(...page.items.filter((item) => !!item));
+            }
+            nextToken = page?.nextToken ?? null;
+        } while (nextToken);
+        return all;
+    }
     async updateComment(input) {
         return this.API.__proto__.UpdateComment(input);
     }
@@ -11668,6 +11796,25 @@ class FeatureService extends BaseObjectService {
         }
         return features;
     }
+    /**
+     * Returns EVERY feature of a space, following `nextToken` until the index is
+     * exhausted, with no user/archive filtering at all.
+     */
+    async getAllFeaturesBySpaceRaw(spaceID) {
+        if (!spaceID) {
+            return [];
+        }
+        const all = [];
+        let nextToken = null;
+        do {
+            const page = await this.API.__proto__.FeaturesBySpace(spaceID, undefined, undefined, 1000, nextToken);
+            if (page?.items?.length > 0) {
+                all.push(...page.items.filter((item) => !!item));
+            }
+            nextToken = page?.nextToken ?? null;
+        } while (nextToken);
+        return all;
+    }
     isBookingPlace(feature) {
         feature.isBookingPlace = feature.type === FeatureType.DESK;
     }
@@ -13714,16 +13861,42 @@ function applyRigidTransformToPoint(p, t) {
         z: t.refTarget.z + (dx * sin + dz * cos)
     };
 }
-/** Applies the Y rotation (in degrees) to an orientation angle (a pose's yRotation). */
+/**
+ * Applies the Y rotation (in degrees) to an orientation angle (a pose's yRotation).
+ * A null/undefined/non-numeric angle is returned untouched: "no orientation stored"
+ * must stay "no orientation stored", not become the raw rotation delta.
+ */
 function applyRigidTransformToYRotationDeg(yRotationDeg, t) {
+    if (yRotationDeg === null || yRotationDeg === undefined || Number.isNaN(Number(yRotationDeg))) {
+        return yRotationDeg;
+    }
     const deltaDeg = t.rotationYRad * (180 / Math.PI);
-    return yRotationDeg + deltaDeg;
+    return Number(yRotationDeg) + deltaDeg;
 }
 /** Euclidean distance between two points, useful to validate landmark quality. */
 function distance(a, b) {
     const dx = a.x - b.x, dy = a.y - b.y, dz = a.z - b.z;
     return Math.sqrt(dx * dx + dy * dy + dz * dz);
 }
+/**
+ * Applies ONLY the rotation part of the transform to a direction vector
+ * (a normal, a stem vector, ...).
+ *
+ * A direction has no origin, so the translation must NOT be applied to it —
+ * doing so would turn a unit normal into a point somewhere in the target model.
+ * Used for a POI's `metadata.normal`, which drives the orientation of the tag's
+ * stem: without this rotation, a pastille copied into a space whose model is
+ * rotated relative to the source ends up pointing into the wall it is stuck on.
+ */
+function applyRigidTransformToDirection(v, t) {
+    const cos = Math.cos(t.rotationYRad);
+    const sin = Math.sin(t.rotationYRad);
+    return {
+        x: v.x * cos - v.z * sin,
+        y: v.y, // vertical axis is not rotated
+        z: v.x * sin + v.z * cos
+    };
+}
 
 /* eslint-disable class-methods-use-this */
 class Object3DService {
@@ -14091,6 +14264,100 @@ i0.ɵɵngDeclareClassMetadata({ minVersion: "12.0.0", version: "17.3.12", ngImpo
                     args: ['currentAPIService']
                 }] }, { type: BaseUserService }, { type: NavigatorService }, { type: ZoneChangeService }, { type: FilterService }] });
 
+class DomainService {
+    API; //AWS
+    constructor(apiInjected) {
+        this.API = apiInjected;
+    }
+    async getGenericDomainsAndForOrganisation(organisationID) {
+        return this.API.__proto__.ListDomains().then((domains) => {
+            return domains.items.filter((dom) => {
+                return (dom.organisationID === organisationID || !dom.organisationID);
+            });
+        });
+    }
+    /**
+     * Same as getGenericDomainsAndForOrganisation(), but drains every page instead of only
+     * the first. `ListDomains` has no per-organisation index — every call already scans the
+     * WHOLE table across every organisation before filtering client-side — so a catalogue
+     * past the first page was silently invisible to callers that need the complete list
+     * (e.g. matching an organisation-specific domain by name across a space-to-space copy).
+     */
+    async getAllGenericDomainsAndForOrganisation(organisationID) {
+        const all = [];
+        let nextToken = null;
+        do {
+            const page = await this.API.__proto__.ListDomains(undefined, 1000, nextToken);
+            if (page?.items?.length > 0) {
+                all.push(...page.items.filter((dom) => !!dom));
+            }
+            nextToken = page?.nextToken ?? null;
+        } while (nextToken);
+        return all.filter(dom => dom.organisationID === organisationID || !dom.organisationID);
+    }
+    async getGenericDomainsAndForOrgAndOrderedOrg(orgID, orderedOrgID = null) {
+        // TODO: Change this (UUID can change depending on DB)
+        if (orgID === "6e28201f-4679-4c29-9d2d-8a325f57cee9") {
+            // I am SmarterPlan, i can see all
+            return this.API.__proto__.ListDomains().then((response) => response.items);
+        }
+        const currentOrgDomains = await this.getGenericDomainsAndForOrganisation(orgID);
+        if (!orderedOrgID || orgID === orderedOrgID) {
+            return currentOrgDomains;
+        }
+        const orderedOrgDomains = await this.getGenericDomainsAndForOrganisation(orderedOrgID);
+        const merged = [...currentOrgDomains, ...orderedOrgDomains];
+        return filterUniqueArrayByID(merged);
+    }
+    async getDomainsForOrgExcludingTypes(types, organisationID) {
+        return this.getGenericDomainsAndForOrganisation(organisationID).then((domains) => {
+            return domains.filter((dom) => !types.includes(dom.type));
+        });
+    }
+    async getDomain(id) {
+        return this.API.__proto__.GetDomain(id);
+    }
+    async updateDomain(input) {
+        return this.API.__proto__.UpdateDomain(input);
+    }
+    async deleteDomain(id) {
+        return this.API.__proto__.DeleteDomain({ id });
+    }
+    async getRootDomains(orgID) {
+        const allDomains = await this.getGenericDomainsAndForOrganisation(orgID);
+        const root = allDomains.filter((dom) => !dom.parentID);
+        return root.sort((a, b) => a.name.localeCompare(b.name));
+    }
+    async createDomain(input) {
+        return this.API.__proto__.CreateDomain(input);
+    }
+    async getDomainsForOrgForType(type, organisationID, orderedOrg = null) {
+        return this.getGenericDomainsAndForOrgAndOrderedOrg(organisationID, orderedOrg).then((domains) => {
+            return domains.filter((dom) => dom.type === type);
+        });
+    }
+    async getParentDomainsForOrgAndOrderedOrgForType(type, organisationID, orderedOrg = null) {
+        return this.getGenericDomainsAndForOrgAndOrderedOrg(organisationID, orderedOrg).then((domains) => {
+            return domains.filter((dom) => dom.type === type && !dom.parentID);
+        });
+    }
+    async getDomainsForMissionsForOrg(orgID) {
+        const domains = await this.getDomainsForOrgExcludingTypes([DomainType.BOOKING, DomainType.FEATURE, DomainType.MISSION], orgID);
+        return domains.filter((dom) => !dom.parentID);
+    }
+    static ɵfac = i0.ɵɵngDeclareFactory({ minVersion: "12.0.0", version: "17.3.12", ngImport: i0, type: DomainService, deps: [{ token: 'currentAPIService' }], target: i0.ɵɵFactoryTarget.Injectable });
+    static ɵprov = i0.ɵɵngDeclareInjectable({ minVersion: "12.0.0", version: "17.3.12", ngImport: i0, type: DomainService, providedIn: "root" });
+}
+i0.ɵɵngDeclareClassMetadata({ minVersion: "12.0.0", version: "17.3.12", ngImport: i0, type: DomainService, decorators: [{
+            type: Injectable,
+            args: [{
+                    providedIn: "root",
+                }]
+        }], ctorParameters: () => [{ type: undefined, decorators: [{
+                    type: Inject,
+                    args: ['currentAPIService']
+                }] }] });
+
 /** Safely extracts a readable message from an unknown catch value, including Amplify's
  *  GraphQL error shape ({ data, errors: [{ errorType, message }] }). */
 function toErrorMessage(err) {
@@ -14142,6 +14409,10 @@ class MatterportTagCopyService {
     featureService;
     measurementService;
     commentService;
+    hashtagService;
+    missionService;
+    operationService;
+    domainService;
     translate;
     // Emits progress updates during copyTagsFromVisit(); null when idle
     copyProgress$ = new BehaviorSubject(null);
@@ -14149,8 +14420,60 @@ class MatterportTagCopyService {
     copyLog$ = new BehaviorSubject([]);
     spModule;
     log = [];
-    errorSeq = 0;
-    constructor(navigationService, userService, poiService, ticketService, equipmentService, object3DService, featureService, measurementService, commentService, translate) {
+    /** Unique-id sequence shared by every permanent (non-step) log line — errors and healed-item notices alike. */
+    logSeq = 0;
+    // ---- Per-run remapping state (reset by resetRunState() at the start of every copy) ----
+    /** source element id -> id of its copy in the target space. */
+    elementIds = {};
+    /** source Comment id -> id of its copy. Needed to remap POI.tagImageID, which is a
+     *  reference to a Comment record (see the schema: "connection to comment.id"). */
+    commentIds = new Map();
+    // Hashtags are space-scoped records, so their ids can never be carried over as-is: they
+    // are matched by NAME against the target space, and the missing ones are created there.
+    // That creation is deliberately lazy — see resolveHashtagIds() — and audited, so an
+    // interrupted copy cannot leave keywords behind that no POI references.
+    /** source Hashtag id -> its name. */
+    sourceHashtagNames = new Map();
+    /** normalised hashtag name -> its id in the TARGET space (pre-existing or just created). */
+    targetHashtagIdsByName = new Map();
+    /** ids of the hashtags this run created in the target space. */
+    createdHashtagIds = new Set();
+    /** ids of the hashtags actually persisted on a successfully created feature. Anything in
+     *  createdHashtagIds but absent here is an orphan and is deleted at the end of the run. */
+    usedHashtagIds = new Set();
+    /** source ids of the elements whose POIs were already created by an element phase.
+     *  Guards the standalone POI phase against creating a second, impoverished copy. */
+    handledElementIds = new Set();
+    // Comment authorship: `Comment.ownerID` is a foreign key to a Mission, and a Mission is
+    // scoped to ONE space (userID + spaceID). A source comment's ownerID therefore points at
+    // nobody in the target space, so it has to be translated to the Mission of the SAME user
+    // there — see resolveCommentOwnerID().
+    /** source Mission id -> its userID. */
+    sourceMissionUserIds = new Map();
+    /** userID -> that user's Mission id in the TARGET space (only set when they have one). */
+    targetMissionIdsByUserId = new Map();
+    // Domain (equipment/ticket/feature/POI category) resolution mirrors the exact rule the
+    // edit forms use to populate their own category dropdown —
+    // DomainService.getDomainsForOrgForType(type, mission.organisationID,
+    // mission.orderedOrganisationID) — keyed to the OPERATOR's own Mission on each space,
+    // NOT to the Space's own organisationID. Two spaces in different organisations can
+    // still share a domain through that Mission link (an operator whose mission on the
+    // target space carries an `orderedOrganisationID` matching the domain's own
+    // organisationID sees — and can reuse — it directly). See resolveDomainID().
+    /** id -> Domain, for every domain visible from EITHER space's operator mission (its
+     *  organisationID, its orderedOrganisationID, or generic/no-organisation domains) — a
+     *  superset covering whatever a source object's domainID could point at. */
+    domainsById = new Map();
+    /** organisationIDs the TARGET operator's mission can see domains from. A domain whose
+     *  own organisationID is in this set (or has none at all) is already directly
+     *  selectable in the target space's own dropdown, so its id is reused unchanged. */
+    targetVisibleDomainOrgIds = new Set();
+    /** `${type}::${normalised name}` -> that domain's id, among domains visible to the
+     *  TARGET operator — used to find an equivalent when the exact id isn't reusable. */
+    targetDomainIdsByTypeAndName = new Map();
+    /** Target space id of the run in progress — used to re-home copied tag icons. */
+    targetSpaceId = null;
+    constructor(navigationService, userService, poiService, ticketService, equipmentService, object3DService, featureService, measurementService, commentService, hashtagService, missionService, operationService, domainService, translate) {
         this.navigationService = navigationService;
         this.userService = userService;
         this.poiService = poiService;
@@ -14160,6 +14483,10 @@ class MatterportTagCopyService {
         this.featureService = featureService;
         this.measurementService = measurementService;
         this.commentService = commentService;
+        this.hashtagService = hashtagService;
+        this.missionService = missionService;
+        this.operationService = operationService;
+        this.domainService = domainService;
         this.translate = translate;
         this.spModule = userService.getSpModule();
     }
@@ -14168,7 +14495,9 @@ class MatterportTagCopyService {
     //
     async copyTagsFromVisit(sourceSpaceId, targetSpaceId, landmarkSourceA, landmarkTargetA, landmarkSourceB, landmarkTargetB) {
         let errorCount = 0;
+        let healedCount = 0;
         this.resetLog();
+        this.resetRunState(targetSpaceId);
         try {
             console.log(`Copying Tags from source ${sourceSpaceId} to target ${targetSpaceId}`);
             this.setStepProgress('collecting', 'collecting', 0);
@@ -14176,7 +14505,7 @@ class MatterportTagCopyService {
                 phase: 'collecting',
                 processedInPhase: 0, totalInPhase: 0,
                 processedOverall: 0, totalOverall: 0,
-                percent: 0, errorCount: 0
+                percent: 0, errorCount: 0, healedCount: 0
             });
             const targetMission = this.userService.currentMission(targetSpaceId);
             const targetMissionId = targetMission ? targetMission.id : null;
@@ -14193,7 +14522,7 @@ class MatterportTagCopyService {
                 this.buildNavigationPositionMap(sourceSpaceId, sourceSweepIds),
                 this.buildNavigationPositionMap(targetSpaceId, targetSweepIds)
             ]);
-            const elementIds = {};
+            const elementIds = this.elementIds;
             let sourceTickets = [];
             let sourceEquipments = [];
             let targetTickets = [];
@@ -14209,9 +14538,31 @@ class MatterportTagCopyService {
             this.setStepProgress('collecting', 'collecting', 55);
             const sourceObjects3D = await this.object3DService.getObjects3DBySpace(sourceSpaceId);
             const targetObjects3D = await this.object3DService.getObjects3DBySpace(targetSpaceId);
-            const sourceFeatures = await this.featureService.getFeaturesBySpaceWithoutFiltering(sourceSpaceId);
-            const targetFeatures = await this.featureService.getFeaturesBySpaceWithoutFiltering(targetSpaceId);
+            // Paginated: a space routinely holds more than the 100 features a single AppSync
+            // page returns, and the tail was silently dropped.
+            //
+            // The source list keeps the historical "skip what the operator archived" rule, so
+            // the set of copied features does not change — but the check is now null-safe:
+            // FeatureService.isArchived() dereferences currentMission(spaceID).id and threw a
+            // TypeError when the operator had no mission on the source space (the common case
+            // for an archived v1 space), aborting the whole copy.
+            // The target list is NOT filtered: it only feeds duplicate detection, and an
+            // archived feature is still a feature that exists there.
+            const sourceFeatures = (await this.featureService.getAllFeaturesBySpaceRaw(sourceSpaceId))
+                .filter(feature => !this.isArchivedForOperator(feature, sourceSpaceId));
+            const targetFeatures = await this.featureService.getAllFeaturesBySpaceRaw(targetSpaceId);
             this.setStepProgress('collecting', 'collecting', 75);
+            // Keywords: index both vocabularies now (read-only). The keywords the target space
+            // is missing are created later, POI by POI, only for those actually used.
+            await this.loadHashtagIndex(sourceSpaceId, targetSpaceId);
+            // Missions: index both spaces now (read-only), so copied comments can credit
+            // their real author instead of whoever is running the copy.
+            await this.loadMissionIndex(sourceSpaceId, targetSpaceId);
+            // Domains: index every domain either operator's mission can see (read-only), so
+            // a category survives a copy across two DIFFERENT organisations whenever it
+            // genuinely would be selectable in the target space's own dropdown — see
+            // loadDomainIndex() for why this is keyed to Mission, not to the Space itself.
+            await this.loadDomainIndex(sourceSpaceId, targetSpaceId);
             if (this.spModule == SpModule.IMMO) {
                 sourceMeasurements = await this.measurementService.getMeasurementsForSpaceForUser(sourceSpaceId);
                 targetMeasurements = await this.measurementService.getMeasurementsForSpaceForUser(targetSpaceId);
@@ -14219,12 +14570,44 @@ class MatterportTagCopyService {
             this.setStepProgress('collecting', 'collecting', 90);
             const sourcePoisAll = await this.poiService.getPoisByMatterportSweeps(sourceSweepIds);
             const targetPoisAll = await this.poiService.getPoisByMatterportSweeps(targetSweepIds);
+            // A POI is "owned" by an element (ticket, equipment, 3D object, feature, measurement)
+            // when its elementID points at one. Those POIs are created by their element's phase,
+            // which is the only place that knows the element's NEW id — so they must be excluded
+            // from the standalone phase, otherwise every feature ends up with two POIs: one
+            // correctly linked but stripped of its tag metadata, and one carrying the metadata
+            // but still pointing at the source space's element.
+            //
+            // Ownership is recognised by TWO independent criteria, and either one is enough to
+            // keep a POI out of the standalone phase:
+            //
+            //  - its type is one an element phase is responsible for. This is the historical
+            //    rule and it must be kept: a POI of that type whose element was NOT collected
+            //    (deleted ticket, equipment outside the current filter, ...) is an orphan, and
+            //    copying it standalone would recreate it with an elementID still pointing into
+            //    the source space — an orphan the previous version correctly refused to copy.
+            //  - its elementID belongs to an element we actually collected. This is the added
+            //    rule, needed because the type alone is not a reliable discriminator: historical
+            //    data also holds INFORMATION/ROOM POIs bound to a feature, which the type test
+            //    lets through and which would end up duplicated.
             const handledPoiTypes = new Set([
                 PoiType.TICKET, PoiType.EQUIPMENT, PoiType.DATA, PoiType.DESK,
                 PoiType.OBJECT3D, PoiType.MEASURE
             ]);
-            const sourcePois = sourcePoisAll.filter(p => !handledPoiTypes.has(p.type));
-            const targetPois = targetPoisAll.filter(p => !handledPoiTypes.has(p.type));
+            for (const owner of [...sourceTickets, ...sourceEquipments, ...sourceObjects3D, ...sourceFeatures]) {
+                if (owner?.id) {
+                    this.handledElementIds.add(owner.id);
+                }
+            }
+            for (const measurement of sourceMeasurements) {
+                // a measurement POI is bound to the measurement's Comment, not to the measurement
+                if (measurement?.comment?.id) {
+                    this.handledElementIds.add(measurement.comment.id);
+                }
+            }
+            const isOwnedByAnElement = (poi) => handledPoiTypes.has(poi.type)
+                || (!!poi.elementID && this.handledElementIds.has(poi.elementID));
+            const sourcePois = sourcePoisAll.filter(p => !isOwnedByAnElement(p));
+            const targetPois = targetPoisAll.filter(p => !!p);
             this.setStepProgress('collecting', 'collecting', 100);
             const totalOverall = sourceTickets.length + sourceEquipments.length + sourcePois.length
                 + sourceObjects3D.length + sourceFeatures.length + sourceMeasurements.length;
@@ -14237,7 +14620,7 @@ class MatterportTagCopyService {
                     processedOverall,
                     totalOverall,
                     percent: totalOverall > 0 ? Math.round((processedOverall / totalOverall) * 100) : 0,
-                    errorCount
+                    errorCount, healedCount
                 });
             };
             if (this.spModule == SpModule.IMMO) {
@@ -14245,8 +14628,8 @@ class MatterportTagCopyService {
                 for (let i = 0; i < sourceTickets.length; i++) {
                     const sourceObj = sourceTickets[i];
                     try {
-                        const existsInTarget = this.matchesExisting(sourceObj, targetTickets);
-                        if (!existsInTarget) {
+                        const existingMatch = this.findExisting(sourceObj, targetTickets, (t) => [t.title, t.description]);
+                        if (!existingMatch) {
                             const rawSource = sourceObj;
                             const ticketInput = {
                                 title: rawSource.title,
@@ -14260,28 +14643,64 @@ class MatterportTagCopyService {
                                 read: rawSource.read || null,
                                 projectID: rawSource.projectID || null,
                                 problemCode: rawSource.problemCode || null,
-                                archivedForMissionID: rawSource.archivedForMissionID || null,
+                                // Missions are per-space: a source mission id designates nobody in
+                                // the target space. Carrying the list over would look like real data
+                                // while hiding the item from no one — reset it, as for features.
+                                archivedForMissionID: null,
                                 spaceID: targetSpaceId,
+                                // Remapped in the second pass below, once every ticket exists.
                                 parentID: null,
-                                domainID: null,
+                                // Reused unchanged in the same organisation, matched by name
+                                // across two — see resolveDomainID().
+                                domainID: this.resolveDomainID(rawSource.domainID),
                                 ownerMissionID: targetMissionId
                             };
                             const createdObj = await this.ticketService.createTicket(ticketInput);
                             if (createdObj && createdObj.id) {
                                 elementIds[sourceObj.id] = createdObj.id;
                             }
-                            if (sourceObj.pois.items && sourceObj.pois.items.length > 0) {
-                                const sourcePoi = sourceObj.pois.items[0];
-                                const targetSweepId = this.findClosestTargetSweepId(sourcePoi.matterportSweepID, sourceNavPositions, targetNavPositions, transform);
-                                const adjustedCoordinate = this.getAdjustedCoordinates(sourcePoi.coordinate, transform);
-                                const poiInput = {
-                                    coordinate: JSON.stringify(adjustedCoordinate),
-                                    type: sourcePoi.type,
-                                    elementID: createdObj.id,
-                                    metadata: sourcePoi.metadata,
-                                    matterportSweepID: targetSweepId
-                                };
-                                await this.poiService.createPOIForSpaceID(poiInput);
+                            // Attachments of the ticket (photos of the incident, documents...).
+                            // They were never copied at all: a duplicated ticket arrived stripped
+                            // of every piece of evidence attached to it.
+                            if (createdObj) {
+                                let sourceComments = [];
+                                try {
+                                    sourceComments = await this.commentService.getAllCommentsForTicket(sourceObj.id);
+                                }
+                                catch (commentsFetchError) {
+                                    console.warn(`Falling back to the embedded comment list for ticket "${rawSource.title}":`, commentsFetchError);
+                                    sourceComments = (sourceObj.comments?.items || []).filter(c => !!c);
+                                }
+                                for (const sourceComment of this.orderCommentsForCopy(sourceComments)) {
+                                    try {
+                                        const created = await this.copyCommentForTicket(sourceComment, createdObj.id, targetSpaceId);
+                                        if (created?.id && sourceComment.id) {
+                                            this.commentIds.set(sourceComment.id, created.id);
+                                        }
+                                    }
+                                    catch (attachmentError) {
+                                        console.error(`Error copying attachment "${sourceComment?.title}" for ticket "${sourceObj?.title}":`, attachmentError);
+                                        errorCount++;
+                                        this.logCopyItemError(attachmentError, 'errorFeatureAttachment', sourceComment?.title || sourceComment?.id, {
+                                            feature: rawSource.title || sourceObj.id
+                                        });
+                                    }
+                                }
+                            }
+                            for (const sourcePoi of (sourceObj.pois?.items || []).filter(p => !!p)) {
+                                await this.createCopiedPoi(sourcePoi, createdObj.id, sourceNavPositions, targetNavPositions, transform);
+                            }
+                        }
+                        else {
+                            // Same reasoning as the EQUIPMENTS phase (see the comment there):
+                            // a ticket matching this one already exists in the target space,
+                            // most likely from an earlier, incomplete run. Fill in whatever
+                            // identity field it is missing rather than leaving it untouched.
+                            elementIds[sourceObj.id] = existingMatch.id;
+                            const healedFields = await this.healTicketIdentity(sourceObj, existingMatch);
+                            if (healedFields.length > 0) {
+                                healedCount++;
+                                this.logHealedItem('ticketHealed', sourceObj?.title || sourceObj?.id, healedFields);
                             }
                         }
                     }
@@ -14295,26 +14714,76 @@ class MatterportTagCopyService {
                     const pct = sourceTickets.length > 0 ? Math.round(((i + 1) / sourceTickets.length) * 100) : 100;
                     this.setStepProgress('tickets', 'tickets', pct);
                 }
-                // EQUIPMENTS (title, POI, PHOTO and NOTES/documents all copied — files physically
-                // duplicated in S3 first, so the copy stays independent from the source)
+                // Second pass: restore the ticket hierarchy. `parentID` had to be nulled during
+                // creation (the parent may not have existed yet) and was then never restored, so
+                // every sub-ticket came out detached. Same rule as for sub-equipments: a parent
+                // with no copied counterpart is dropped rather than left pointing at the source
+                // space's ticket.
+                for (const sourceObj of sourceTickets) {
+                    const rawSource = sourceObj;
+                    const newId = elementIds[sourceObj.id];
+                    const newParentId = rawSource.parentID ? elementIds[rawSource.parentID] : null;
+                    if (newId && newParentId) {
+                        try {
+                            await this.ticketService.updateTicket({ id: newId, parentID: newParentId });
+                        }
+                        catch (parentError) {
+                            console.error(`Error remapping the parent ticket of "${sourceObj?.title}":`, parentError);
+                            errorCount++;
+                            this.logCopyItemError(parentError, 'errorTicket', sourceObj?.title || sourceObj?.id);
+                        }
+                    }
+                }
+                // EQUIPMENTS (identity, POI, PHOTO and NOTES/documents all copied — files
+                // physically duplicated in S3 first, so the copy stays independent from the source)
+                //
+                // CAREFUL: the Equipment model has NO `title` and NO `description` column. It
+                // identifies itself with `name` (String! — required) and annotates itself with
+                // `annotation`. Reading `rawSource.title` therefore always yielded undefined and
+                // the required `name` was never sent, which AppSync rejected with
+                // "Variable 'input' has coerced Null value for NonNull type 'String!'".
                 for (let i = 0; i < sourceEquipments.length; i++) {
                     const sourceObj = sourceEquipments[i];
                     try {
-                        const existsInTarget = this.matchesExisting(sourceObj, targetEquipments);
-                        if (!existsInTarget) {
+                        // Equipment has no `title`/`description`: it identifies itself by name, and by
+                        // serial number when one is recorded.
+                        const existingMatch = this.findExisting(sourceObj, targetEquipments, (e) => [e.name, e.serialNumber]);
+                        if (!existingMatch) {
                             const rawSource = sourceObj;
                             let copiedPhotoKey = null;
                             if (rawSource.photo) {
-                                copiedPhotoKey = await this.copyS3KeyIfPresent(rawSource.photo, `equipment photo "${rawSource.title}"`);
+                                copiedPhotoKey = await this.copyS3KeyIfPresent(rawSource.photo, `equipment photo "${rawSource.name}"`);
                             }
                             const equipmentInput = {
-                                title: rawSource.title,
+                                name: rawSource.name,
+                                // Unlike every other model this file copies, Equipment declares
+                                // `createdAt` as a plain client-settable field rather than an
+                                // AppSync-managed timestamp (see schema.graphql) — so, uniquely
+                                // here, the original commissioning date can actually be kept
+                                // instead of the copy silently getting today's date.
+                                createdAt: rawSource.createdAt ?? null,
+                                annotation: rawSource.annotation ?? null,
                                 status: rawSource.status,
-                                description: rawSource.description,
-                                archivedForMissionID: rawSource.archivedForMissionID || null,
+                                // Full identity of the equipment — all of these were dropped
+                                family: rawSource.family ?? null,
+                                sku: rawSource.sku ?? null,
+                                serialNumber: rawSource.serialNumber ?? null,
+                                manufacturer: rawSource.manufacturer ?? null,
+                                installDate: rawSource.installDate ?? null,
+                                warrantyEndDate: rawSource.warrantyEndDate ?? null,
+                                metadata: rawSource.metadata ?? null,
+                                isCreatedByAI: rawSource.isCreatedByAI ?? null,
+                                // Missions are per-space: a source mission id designates nobody in
+                                // the target space. Carrying the list over would look like real data
+                                // while hiding the item from no one — reset it, as for features.
+                                archivedForMissionID: null,
                                 spaceID: targetSpaceId,
-                                domainID: null,
+                                // Reused unchanged in the same organisation, matched by name
+                                // across two — see resolveDomainID().
+                                domainID: this.resolveDomainID(rawSource.domainID),
                                 photo: copiedPhotoKey,
+                                // `subequipmentsIDs` is deliberately absent: it is remapped to the
+                                // copied equipments in the second pass, once they all exist.
                             };
                             const createdObj = await this.equipmentService.createEquipment(equipmentInput, null, null);
                             if (createdObj && createdObj.id) {
@@ -14324,38 +14793,59 @@ class MatterportTagCopyService {
                             if (createdObj && rawSource.notes?.items && rawSource.notes.items.length > 0) {
                                 for (const sourceNote of rawSource.notes.items) {
                                     try {
-                                        if (!sourceNote.attachment) {
-                                            continue; // nothing to duplicate for this note
-                                        }
-                                        const copiedKey = await this.copyS3KeyIfPresent(sourceNote.attachment, `equipment note for "${sourceObj?.title}"`);
-                                        await this.equipmentService.createNoteWithExistingAttachment(createdObj.id, copiedKey, sourceNote.type, targetMissionId);
+                                        // A note without a file is still a note: it carries a type
+                                        // and its signature/expiry dates. Skipping it, as was done
+                                        // before, silently dropped those records entirely.
+                                        const copiedKey = sourceNote.attachment
+                                            ? await this.copyS3KeyIfPresent(sourceNote.attachment, `equipment note for "${sourceObj?.name}"`)
+                                            : null;
+                                        await this.equipmentService.createNoteWithExistingAttachment(createdObj.id, copiedKey, sourceNote.type, targetMissionId, {
+                                            signDate: sourceNote.signDate ?? null,
+                                            expirationDate: sourceNote.expirationDate ?? null
+                                        });
                                     }
                                     catch (noteError) {
-                                        console.error(`Error copying note for equipment "${sourceObj?.title}":`, noteError);
+                                        console.error(`Error copying note for equipment "${sourceObj?.name}":`, noteError);
                                         errorCount++;
-                                        this.logCopyItemError(noteError, 'errorEquipmentNote', sourceObj?.title || sourceObj?.id);
+                                        this.logCopyItemError(noteError, 'errorEquipmentNote', sourceObj?.name || sourceObj?.id, {
+                                            equipment: sourceObj?.name || sourceObj?.id
+                                        });
                                     }
                                 }
                             }
-                            if (sourceObj.pois.items && sourceObj.pois.items.length > 0) {
-                                const sourcePoi = sourceObj.pois.items[0];
-                                const targetSweepId = this.findClosestTargetSweepId(sourcePoi.matterportSweepID, sourceNavPositions, targetNavPositions, transform);
-                                const adjustedCoordinate = this.getAdjustedCoordinates(sourcePoi.coordinate, transform);
-                                const poiInput = {
-                                    coordinate: JSON.stringify(adjustedCoordinate),
-                                    type: sourcePoi.type,
-                                    elementID: createdObj.id,
-                                    metadata: sourcePoi.metadata,
-                                    matterportSweepID: targetSweepId
-                                };
-                                await this.poiService.createPOIForSpaceID(poiInput);
+                            for (const sourcePoi of (sourceObj.pois?.items || []).filter(p => !!p)) {
+                                await this.createCopiedPoi(sourcePoi, createdObj.id, sourceNavPositions, targetNavPositions, transform);
+                            }
+                        }
+                        else {
+                            // An equipment matching this one already exists in the target space —
+                            // most often because a PREVIOUS run of this copy already created it.
+                            // That earlier run may have been interrupted, or hit a bug that has
+                            // since been fixed (missing family/manufacturer/domainID/... were once
+                            // dropped entirely — see the comment above), leaving an equipment with
+                            // its identity fields empty forever: re-running the copy used to skip
+                            // it outright, since it already "exists", so the gap could never heal.
+                            //
+                            // Re-running the tool must be able to repair that: any identity field
+                            // the existing target equipment is missing, and the source has, is
+                            // filled in. Fields the target already carries a value for are left
+                            // untouched, so a manual edit made in the target space is never
+                            // overwritten by an older or different source value. Notes and POIs are
+                            // NOT re-created here: unlike scalar fields they cannot be "missing vs.
+                            // present" checked the same way, and re-adding them on every re-run of
+                            // an already-healthy copy would duplicate them.
+                            elementIds[sourceObj.id] = existingMatch.id;
+                            const healedFields = await this.healEquipmentIdentity(sourceObj, existingMatch);
+                            if (healedFields.length > 0) {
+                                healedCount++;
+                                this.logHealedItem('equipmentHealed', sourceObj?.name || sourceObj?.id, healedFields);
                             }
                         }
                     }
                     catch (itemError) {
-                        console.error(`Error copying equipment "${sourceObj?.title}" (id: ${sourceObj?.id}):`, itemError);
+                        console.error(`Error copying equipment "${sourceObj?.name}" (id: ${sourceObj?.id}):`, itemError);
                         errorCount++;
-                        this.logCopyItemError(itemError, 'errorEquipment', sourceObj?.title || sourceObj?.id);
+                        this.logCopyItemError(itemError, 'errorEquipment', sourceObj?.name || sourceObj?.id);
                     }
                     processedOverall++;
                     emitProgress('equipments', i + 1, sourceEquipments.length);
@@ -14378,53 +14868,59 @@ class MatterportTagCopyService {
                                 await this.equipmentService.updateEquipment({ id: newId, subequipmentsIDs: remapped });
                             }
                             catch (subEquipError) {
-                                console.error(`Error remapping sub-equipments for "${sourceObj?.title}":`, subEquipError);
+                                console.error(`Error remapping sub-equipments for "${sourceObj?.name}":`, subEquipError);
                                 errorCount++;
-                                this.logCopyItemError(subEquipError, 'errorEquipmentSubEquip', sourceObj?.title || sourceObj?.id);
+                                this.logCopyItemError(subEquipError, 'errorEquipmentSubEquip', sourceObj?.name || sourceObj?.id, {
+                                    equipment: sourceObj?.name || sourceObj?.id
+                                });
                             }
                         }
                     }
                 }
-            }
-            // POIS (standalone — types already handled above are excluded)
-            for (let i = 0; i < sourcePois.length; i++) {
-                const sourcePoi = sourcePois[i];
-                try {
-                    const existsInTarget = targetPois.some(tp => tp.coordinate === sourcePoi.coordinate && tp.tagDescription === sourcePoi.tagDescription);
-                    if (!existsInTarget) {
-                        const targetSweepId = this.findClosestTargetSweepId(sourcePoi.matterportSweepID, sourceNavPositions, targetNavPositions, transform);
-                        const adjustedCoordinate = this.getAdjustedCoordinates(sourcePoi.coordinate, transform);
-                        let elementID = sourcePoi.elementID;
-                        if (elementID && elementIds[elementID]) {
-                            elementID = elementIds[elementID];
+                // OPERATIONS — the Ticket<->Equipment join (drives "Tickets actuels" on the
+                // equipment page and the linked-equipment list on the ticket page). Both the
+                // TICKETS and EQUIPMENTS phases above copy the record itself, but neither one
+                // touches this join table, so every link between a copied ticket and a copied
+                // equipment was silently dropped: both sides existed in the target space, but
+                // pointing at nobody.
+                //
+                // Runs after both phases, over every SOURCE ticket that ended up mapped in
+                // the target space (freshly created OR healed — elementIds covers both), so a
+                // re-run over an already-copied pair of spaces can still pick up links that
+                // an earlier, incomplete run never created.
+                for (const sourceTicket of sourceTickets) {
+                    const newTicketId = elementIds[sourceTicket.id];
+                    if (!newTicketId) {
+                        continue;
+                    }
+                    try {
+                        const sourceOps = await this.operationService.getOperationsByTicketID(sourceTicket.id);
+                        if (sourceOps.length === 0) {
+                            continue;
+                        }
+                        // Read what the target ticket is already linked to, so a re-run never
+                        // creates a duplicate Operation for a link that already exists.
+                        const targetOps = await this.operationService.getOperationsByTicketID(newTicketId);
+                        const linkedEquipmentIds = new Set(targetOps.map(op => op.equipmentID));
+                        for (const sourceOp of sourceOps) {
+                            const newEquipmentId = sourceOp.equipmentID ? elementIds[sourceOp.equipmentID] : null;
+                            // No copied counterpart for the source equipment (not collected,
+                            // outside the current filter, ...) — same rule as everywhere else
+                            // in this file: drop the link rather than point it at nothing or
+                            // at the source space's equipment.
+                            if (!newEquipmentId || linkedEquipmentIds.has(newEquipmentId)) {
+                                continue;
+                            }
+                            await this.operationService.createOperation({ ticketID: newTicketId, equipmentID: newEquipmentId });
+                            linkedEquipmentIds.add(newEquipmentId);
                         }
-                        const copiedTagIcon = await this.duplicateTagIcon(sourcePoi.tagIcon);
-                        const copiedTagImageID = await this.duplicateTagImageID(sourcePoi.tagImageID);
-                        const newPoi = {
-                            coordinate: JSON.stringify(adjustedCoordinate),
-                            type: sourcePoi.type,
-                            elementID: elementID,
-                            tagDescription: sourcePoi.tagDescription,
-                            tagIcon: copiedTagIcon,
-                            tagImageID: copiedTagImageID,
-                            virtual: sourcePoi.virtual,
-                            xRotation: sourcePoi.xRotation,
-                            yRotation: sourcePoi.yRotation,
-                            metadata: sourcePoi.metadata,
-                            matterportSweepID: targetSweepId
-                        };
-                        await this.poiService.createPOIForSpaceID(newPoi);
+                    }
+                    catch (opError) {
+                        console.error(`Error copying ticket-equipment links for "${sourceTicket?.title}":`, opError);
+                        errorCount++;
+                        this.logCopyItemError(opError, 'errorOperation', sourceTicket?.title || sourceTicket?.id);
                     }
                 }
-                catch (itemError) {
-                    console.error(`Error copying POI (sweep: ${sourcePoi?.matterportSweepID}):`, itemError);
-                    errorCount++;
-                    this.logCopyItemError(itemError, 'errorPoi', sourcePoi?.matterportSweepID);
-                }
-                processedOverall++;
-                emitProgress('pois', i + 1, sourcePois.length);
-                const pct = sourcePois.length > 0 ? Math.round(((i + 1) / sourcePois.length) * 100) : 100;
-                this.setStepProgress('pois', 'pois', pct);
             }
             // OBJECTS 3D (the 3D model file itself — `path` — is physically duplicated in S3)
             for (let i = 0; i < sourceObjects3D.length; i++) {
@@ -14433,8 +14929,16 @@ class MatterportTagCopyService {
                     const existsInTarget = targetObjects3D.some(targetObj => targetObj.name === sourceObj.name);
                     if (!existsInTarget) {
                         const rawSource = sourceObj;
+                        // `path` means two different things depending on `s3Bucket`, exactly as the
+                        // viewer reads it (see viewer.component.ts and matterport-object3d.service.ts):
+                        //  - s3Bucket === true  -> a key of a model file uploaded to S3, ours to duplicate;
+                        //  - otherwise          -> a reference into the 3D object library shipped with
+                        //    the application ("3DObjects/objects/cube.dae" -> assets/3Dobjects/...),
+                        //    which exists in every space and must be reused verbatim.
+                        // Copying unconditionally meant every library object failed with NoSuchKey and
+                        // the whole 3D object was skipped, model AND POI.
                         let copiedPath = rawSource.path;
-                        if (rawSource.path) {
+                        if (rawSource.path && rawSource.s3Bucket) {
                             copiedPath = await this.copyS3KeyIfPresent(rawSource.path, `3D object "${rawSource.name}"`);
                         }
                         /**
@@ -14464,39 +14968,65 @@ class MatterportTagCopyService {
                             // Ensure we return valid numbers
                             return array.map(n => isNaN(n) ? 0 : n);
                         };
+                        // This service is shared, but the two applications do NOT deploy the same
+                        // Object3D schema: `viewFrustum` exists only in smarterplan, and `scale`
+                        // is [Float] there but a plain Float in museum. Sending the smarterplan
+                        // shape to museum was rejected with "The variables input contains a field
+                        // that is not defined for input object type 'CreateObject3DInput'".
+                        //
+                        // The input therefore MIRRORS the source record instead of imposing a
+                        // shape: source and target live in the same deployment, so whatever the
+                        // source carries is exactly what the target accepts.
                         const object3DInput = {
                             path: copiedPath,
+                            // Tells the viewer how to interpret `path`. Dropping it turned an
+                            // S3-hosted model into a library reference on the copy, which the
+                            // viewer then tried to resolve against assets/ and failed to load.
+                            s3Bucket: !!rawSource.s3Bucket,
                             name: rawSource.name,
                             description: rawSource.description || "",
-                            viewFrustum: !!rawSource.viewFrustum,
                             spaceID: targetSpaceId,
                             position: toFloatArray(rawSource.position, [0, 0, 0]),
-                            rotation: toFloatArray(rawSource.rotation, [0, 0, 0]),
-                            // Enforce scale to be at least 1.0 if it's invalid
-                            scale: toFloatArray(rawSource.scale, [1, 1, 1])
+                            rotation: toFloatArray(rawSource.rotation, [0, 0, 0])
                         };
-                        // Debugging: Inspect the exact object sent to the mutation
-                        console.log(`[DEBUG] Sending Object3D "${rawSource.name}" to API:`, JSON.stringify(object3DInput));
+                        if ('viewFrustum' in rawSource) {
+                            object3DInput.viewFrustum = !!rawSource.viewFrustum;
+                        }
+                        // Scale keeps the source's own shape: a list stays a list, a scalar stays
+                        // a scalar. An absent scale is left unset rather than invented, so the copy
+                        // renders exactly like the original.
+                        if (Array.isArray(rawSource.scale)) {
+                            object3DInput.scale = toFloatArray(rawSource.scale, [1, 1, 1]);
+                        }
+                        else if (rawSource.scale !== null && rawSource.scale !== undefined) {
+                            const scalarScale = parseFloat(rawSource.scale);
+                            object3DInput.scale = Number.isNaN(scalarScale) ? 1 : scalarScale;
+                        }
                         const createdObj = await this.object3DService.create(object3DInput);
                         if (createdObj && createdObj.id) {
                             elementIds[sourceObj.id] = createdObj.id;
                         }
-                        if (sourceObj.pois?.items && sourceObj.pois.items.length > 0) {
-                            const sourcePoi = sourceObj.pois.items[0];
-                            const targetSweepId = this.findClosestTargetSweepId(sourcePoi.matterportSweepID, sourceNavPositions, targetNavPositions, transform);
-                            const adjustedCoordinate = this.getAdjustedCoordinates(sourcePoi.coordinate, transform);
-                            const poiInput = {
-                                coordinate: JSON.stringify(adjustedCoordinate),
-                                type: sourcePoi.type,
-                                elementID: createdObj.id,
-                                metadata: sourcePoi.metadata,
-                                matterportSweepID: targetSweepId
-                            };
-                            await this.poiService.createPOIForSpaceID(poiInput);
+                        for (const sourcePoi of (sourceObj.pois?.items || []).filter(p => !!p)) {
+                            await this.createCopiedPoi(sourcePoi, createdObj.id, sourceNavPositions, targetNavPositions, transform);
                         }
                     }
                 }
                 catch (itemError) {
+                    // The backend can deny creation of this whole model to the signed-in account
+                    // (in MUSEUM, Object3D only grants get/list to userPools). The denial is not
+                    // per-item, so every remaining object would fail identically: report it once,
+                    // as a permission problem rather than a copy error, and stop the phase instead
+                    // of firing one doomed mutation per object.
+                    if (this.isUnauthorized(itemError)) {
+                        const remaining = sourceObjects3D.length - i;
+                        console.warn(`[MatterportTagCopyService] Not allowed to create 3D objects in the target space, skipping ${remaining} of them.`, itemError);
+                        errorCount++;
+                        this.logItemError('errorObjects3dNotAllowed', { count: remaining });
+                        processedOverall += remaining;
+                        emitProgress('objects3d', sourceObjects3D.length, sourceObjects3D.length);
+                        this.setStepProgress('objects3d', 'objects3d', 100);
+                        break;
+                    }
                     console.error(`Error copying object3D "${sourceObj?.name}":`, itemError);
                     errorCount++;
                     this.logCopyItemError(itemError, 'errorObject3d', sourceObj?.name || sourceObj?.id);
@@ -14506,76 +15036,98 @@ class MatterportTagCopyService {
                 const pct = sourceObjects3D.length > 0 ? Math.round(((i + 1) / sourceObjects3D.length) * 100) : 100;
                 this.setStepProgress('objects3d', 'objects3d', pct);
             }
-            // FEATURES (title, POI, and attached comments — images/videos/docs/links — all copied,
-            // each file physically duplicated in S3 first)
+            // FEATURES — the "POI" of the MUSEUM module: its own record (title, index, locale,
+            // keywords), its media set (one Comment per image/video/audio/document/link/text)
+            // and its POI record (the pastille: icon, tag description, cover image).
+            // Everything is copied here, files physically duplicated in S3 first.
             for (let i = 0; i < sourceFeatures.length; i++) {
                 const sourceObj = sourceFeatures[i];
                 try {
-                    const existsInTarget = this.matchesExisting(sourceObj, targetFeatures);
-                    if (!existsInTarget) {
+                    const existingMatch = this.findExisting(sourceObj, targetFeatures, (f) => [f.title, f.description]);
+                    if (!existingMatch) {
                         const rawSource = sourceObj;
+                        // Resolved just before the write: a keyword is only created in the target
+                        // space when the POI carrying it is about to be created.
+                        const resolvedHashtags = await this.resolveHashtagIds(rawSource.hashtags, targetSpaceId);
                         const featureInput = {
                             title: rawSource.title,
                             shortDescription: rawSource.shortDescription,
                             level: rawSource.level,
                             type: rawSource.type,
-                            archivedForMissionID: rawSource.archivedForMissionID || null,
+                            // `index` drives the ordering of the POI in the space's list, in the
+                            // lateral menu and in the guided tour. Dropping it forced a full manual
+                            // renumbering of every copied POI.
+                            index: rawSource.index ?? null,
+                            // Missions are per-space: source mission ids mean nothing in the target
+                            // space, so an archive list carried over would hide POIs from nobody
+                            // while looking like real data. Reset instead.
+                            archivedForMissionID: null,
                             stripeProductID: rawSource.stripeProductID || null,
                             stripePriceID: rawSource.stripePriceID || null,
                             bookingPriceMetadata: rawSource.bookingPriceMetadata || null,
                             locale: rawSource.locale || null,
-                            hashtags: rawSource.hashtags || null,
+                            // Hashtags are space-scoped records: the ids must be translated to the
+                            // equivalent hashtags of the target space (created on demand).
+                            hashtags: resolvedHashtags,
                             description: rawSource.description || null,
-                            domainID: null,
+                            // Reused unchanged in the same organisation, matched by name
+                            // across two — see resolveDomainID().
+                            domainID: this.resolveDomainID(rawSource.domainID),
                             spaceID: targetSpaceId,
                             ownerMissionID: targetMissionId
                         };
                         const createdObj = await this.featureService.createFeature(featureInput);
                         if (createdObj && createdObj.id) {
                             elementIds[sourceObj.id] = createdObj.id;
+                            // The feature exists: its keywords are now genuinely referenced and
+                            // must survive the end-of-run cleanup.
+                            this.markHashtagsUsed(resolvedHashtags);
                         }
-                        if (createdObj && sourceObj.comments?.items && sourceObj.comments.items.length > 0) {
-                            for (const sourceComment of sourceObj.comments.items) {
+                        if (createdObj) {
+                            // Read the media set through its own paginated query: the `comments`
+                            // connection embedded in FeaturesBySpace is capped at 100 items and its
+                            // nextToken was never followed, so rich POIs lost attachments silently.
+                            let sourceComments = [];
+                            try {
+                                sourceComments = await this.commentService.getAllCommentsForFeature(sourceObj.id);
+                            }
+                            catch (commentsFetchError) {
+                                console.warn(`Falling back to the embedded comment list for feature "${rawSource.title}":`, commentsFetchError);
+                                sourceComments = (sourceObj.comments?.items || []).filter(c => !!c);
+                            }
+                            for (const sourceComment of this.orderCommentsForCopy(sourceComments)) {
                                 try {
-                                    const rawComment = sourceComment;
-                                    let copiedAnnexe = null;
-                                    if (rawComment.annexe) {
-                                        copiedAnnexe = await this.copyS3KeyIfPresent(rawComment.annexe, `feature attachment "${rawComment.title}"`);
+                                    const created = await this.copyCommentForFeature(sourceComment, createdObj.id, targetSpaceId);
+                                    if (created?.id && sourceComment.id) {
+                                        this.commentIds.set(sourceComment.id, created.id);
                                     }
-                                    const commentInput = {
-                                        title: rawComment.title,
-                                        type: rawComment.type,
-                                        annexe: copiedAnnexe,
-                                        externalLink: rawComment.externalLink || null,
-                                        dimensions: rawComment.dimensions || null,
-                                        index: rawComment.index,
-                                        shownInTag: rawComment.shownInTag,
-                                        isCoverImage: rawComment.isCoverImage,
-                                        featureID: createdObj.id,
-                                        spaceID: targetSpaceId,
-                                        ownerID: targetMissionId
-                                    };
-                                    await this.commentService.createComment(commentInput);
                                 }
                                 catch (attachmentError) {
                                     console.error(`Error copying attachment "${sourceComment?.title}" for feature "${sourceObj?.title}":`, attachmentError);
                                     errorCount++;
-                                    this.logCopyItemError(attachmentError, 'errorFeatureAttachment', sourceComment?.title || sourceComment?.id);
+                                    this.logCopyItemError(attachmentError, 'errorFeatureAttachment', sourceComment?.title || sourceComment?.id, {
+                                        feature: rawSource.title || sourceObj.id
+                                    });
                                 }
                             }
+                            // Every POI of the feature, not only the first one: a feature may be
+                            // pinned at several places, and only this phase knows the new element id.
+                            const sourceFeaturePois = (sourceObj.pois?.items || []).filter(p => !!p);
+                            for (const sourcePoi of sourceFeaturePois) {
+                                await this.createCopiedPoi(sourcePoi, createdObj.id, sourceNavPositions, targetNavPositions, transform);
+                            }
                         }
-                        if (sourceObj.pois?.items && sourceObj.pois.items.length > 0) {
-                            const sourcePoi = sourceObj.pois.items[0];
-                            const targetSweepId = this.findClosestTargetSweepId(sourcePoi.matterportSweepID, sourceNavPositions, targetNavPositions, transform);
-                            const adjustedCoordinate = this.getAdjustedCoordinates(sourcePoi.coordinate, transform);
-                            const poiInput = {
-                                coordinate: JSON.stringify(adjustedCoordinate),
-                                type: sourcePoi.type,
-                                elementID: createdObj.id,
-                                metadata: sourcePoi.metadata,
-                                matterportSweepID: targetSweepId
-                            };
-                            await this.poiService.createPOIForSpaceID(poiInput);
+                    }
+                    else {
+                        // Same reasoning as the EQUIPMENTS phase (see the comment there): a
+                        // feature matching this one already exists in the target space, most
+                        // likely from an earlier, incomplete run. Fill in whatever identity
+                        // field it is missing rather than leaving it untouched.
+                        elementIds[sourceObj.id] = existingMatch.id;
+                        const healedFields = await this.healFeatureIdentity(sourceObj, existingMatch);
+                        if (healedFields.length > 0) {
+                            healedCount++;
+                            this.logHealedItem('featureHealed', sourceObj?.title || sourceObj?.id, healedFields);
                         }
                     }
                 }
@@ -14604,12 +15156,20 @@ class MatterportTagCopyService {
                             }
                             const commentInput = {
                                 title: rawComment.title,
+                                description: rawComment.description ?? null,
+                                metadata: rawComment.metadata ?? null,
                                 type: CommentType.MEASURE,
                                 dimensions: rawComment.dimensions || null,
                                 annexe: copiedAnnexe,
                                 spaceID: targetSpaceId,
+                                // Same authorship rule as every other copied comment — see
+                                // resolveCommentOwnerID().
+                                ownerID: this.resolveCommentOwnerID(rawComment.ownerID),
                             };
                             const createdComment = await this.commentService.createComment(commentInput);
+                            if (createdComment && createdComment.id && rawComment.id) {
+                                this.commentIds.set(rawComment.id, createdComment.id);
+                            }
                             if (createdComment && sourceMeasurement.poi) {
                                 const targetSweepId = this.findClosestTargetSweepId(sourceMeasurement.poi.matterportSweepID, sourceNavPositions, targetNavPositions, transform);
                                 const adjustedPosition = this.getAdjustedPositionOnly(sourceMeasurement.poi.coordinate, transform);
@@ -14617,7 +15177,9 @@ class MatterportTagCopyService {
                                     coordinate: JSON.stringify(adjustedPosition),
                                     type: PoiType.MEASURE,
                                     elementID: createdComment.id,
-                                    metadata: sourceMeasurement.poi.metadata,
+                                    // Same treatment as any other POI: the source's Matterport tag
+                                    // SID must not travel, and the stem normal has to be rotated.
+                                    metadata: this.getAdjustedPoiMetadata(sourceMeasurement.poi.metadata, transform),
                                     matterportSweepID: targetSweepId
                                 };
                                 await this.poiService.createPOIForSpaceID(poiInput);
@@ -14635,15 +15197,51 @@ class MatterportTagCopyService {
                     this.setStepProgress('measurements', 'measurements', pct);
                 }
             }
+            // POIS — standalone ones only: every POI owned by an element was already created by
+            // that element's phase, with the element's NEW id.
+            //
+            // This phase runs LAST on purpose. It used to run third, before 3D objects, features
+            // and measurements existed in the target space, so `elementIds` and `commentIds` were
+            // still empty when it needed them: elementID stayed pointing at the source space and
+            // tagImageID (a reference to a Comment record) could never be resolved.
+            for (let i = 0; i < sourcePois.length; i++) {
+                const sourcePoi = sourcePois[i];
+                try {
+                    const existsInTarget = targetPois.some(tp => tp.coordinate === sourcePoi.coordinate && tp.tagDescription === sourcePoi.tagDescription);
+                    if (!existsInTarget) {
+                        const elementID = sourcePoi.elementID
+                            ? (elementIds[sourcePoi.elementID] || sourcePoi.elementID)
+                            : sourcePoi.elementID;
+                        await this.createCopiedPoi(sourcePoi, elementID, sourceNavPositions, targetNavPositions, transform);
+                    }
+                }
+                catch (itemError) {
+                    console.error(`Error copying POI (sweep: ${sourcePoi?.matterportSweepID}):`, itemError);
+                    errorCount++;
+                    this.logCopyItemError(itemError, 'errorPoi', sourcePoi?.matterportSweepID, {
+                        sweep: sourcePoi?.matterportSweepID
+                    });
+                }
+                processedOverall++;
+                emitProgress('pois', i + 1, sourcePois.length);
+                const pct = sourcePois.length > 0 ? Math.round(((i + 1) / sourcePois.length) * 100) : 100;
+                this.setStepProgress('pois', 'pois', pct);
+            }
             this.copyProgress$.next({
                 phase: 'done',
                 processedInPhase: 0, totalInPhase: 0,
                 processedOverall, totalOverall,
-                percent: 100, errorCount
+                percent: 100, errorCount, healedCount
             });
             this.setStepLog('done', errorCount > 0
                 ? this.translate.instant('matterportCopy.doneWithErrors', { count: errorCount })
                 : this.translate.instant('matterportCopy.done'));
+            // Separate summary line, on top of (not instead of) the done/doneWithErrors one
+            // above: healing is orthogonal to success/failure, an already-complete run can
+            // still have repaired items left over from an earlier, incomplete attempt.
+            if (healedCount > 0) {
+                this.setStepLog('healedSummary', this.translate.instant('matterportCopy.healedSummary', { count: healedCount }));
+            }
             return true;
         }
         catch (error) {
@@ -14652,6 +15250,23 @@ class MatterportTagCopyService {
             this.copyProgress$.next(null);
             return false;
         }
+        finally {
+            // Every exit path — success, partial success, or general failure — gives back the
+            // keywords this run created but no copied POI ended up using. Placed in `finally`
+            // so an exception thrown mid-copy cannot leave them behind either.
+            await this.cleanupUnusedCreatedHashtags();
+        }
+    }
+    /**
+     * Serialises the current copy log (step lines as last emitted, plus every error and
+     * healed-item notice) to a plain-text file and triggers a browser download — wired to a
+     * "download log" button in the copy UI, so an operator can keep or share the full report
+     * of a run rather than only the ~200px scrollable panel showing it.
+     */
+    downloadLog(filename) {
+        const text = this.log.map(entry => entry.text).join('\n');
+        const defaultName = `matterport-copy-log-${new Date().toISOString().replace(/[:.]/g, '-')}.txt`;
+        downloadBlob(new Blob([text], { type: 'text/plain;charset=utf-8' }), filename || defaultName);
     }
     //
     // ---------- Progress / log helpers ----------
@@ -14670,16 +15285,41 @@ class MatterportTagCopyService {
     }
     /** Appends a new, permanent log line (never overwritten), then republishes the log. */
     appendErrorLog(text) {
-        this.errorSeq++;
-        this.log.push({ id: `error-${this.errorSeq}`, type: 'error', text });
+        this.logSeq++;
+        this.log.push({ id: `error-${this.logSeq}`, type: 'error', text });
+        this.copyLog$.next([...this.log]);
+    }
+    /** Appends a permanent log line for an item that was repaired rather than newly
+     *  created — see healEquipmentIdentity() and its ticket/feature equivalents. */
+    appendHealedLog(text) {
+        this.logSeq++;
+        this.log.push({ id: `healed-${this.logSeq}`, type: 'healed', text });
         this.copyLog$.next([...this.log]);
     }
     /** Resets the log for a new run. */
     resetLog() {
         this.log = [];
-        this.errorSeq = 0;
+        this.logSeq = 0;
         this.copyLog$.next([]);
     }
+    /** Clears every id map carried over from a previous run. The service is a root-level
+     *  singleton, so leftover mappings would silently link a new copy to records created
+     *  during the previous one. */
+    resetRunState(targetSpaceId) {
+        this.elementIds = {};
+        this.commentIds = new Map();
+        this.sourceHashtagNames = new Map();
+        this.targetHashtagIdsByName = new Map();
+        this.createdHashtagIds = new Set();
+        this.usedHashtagIds = new Set();
+        this.handledElementIds = new Set();
+        this.sourceMissionUserIds = new Map();
+        this.targetMissionIdsByUserId = new Map();
+        this.domainsById = new Map();
+        this.targetVisibleDomainOrgIds = new Set();
+        this.targetDomainIdsByTypeAndName = new Map();
+        this.targetSpaceId = targetSpaceId;
+    }
     /** Builds and sets a step's log line as "<translated label>: <percent>%", no step numbering. */
     setStepProgress(stepId, translationKey, percent) {
         const label = this.translate.instant(`matterportCopy.${translationKey}`);
@@ -14690,31 +15330,628 @@ class MatterportTagCopyService {
         const text = this.translate.instant(`matterportCopy.${translationKey}`, params);
         this.appendErrorLog(text);
     }
+    /** Logs a translated line for an item that already existed in the target space and had
+     *  its missing identity fields filled in, so the operator can tell it apart from a
+     *  freshly created item in the copy report. */
+    logHealedItem(translationKey, itemLabel, healedFields) {
+        const text = this.translate.instant(`matterportCopy.${translationKey}`, {
+            name: itemLabel,
+            fields: healedFields.join(', ')
+        });
+        this.appendHealedLog(text);
+    }
     /** Logs an item-level error, using a specific "missing file / orphaned reference"
      *  translation key when the cause is a MissingS3FileError, and the generic per-type
      *  key otherwise. Keeps the copy log readable: a missing file is not a bug to
-     *  investigate the same way a network/permission error is. */
-    logCopyItemError(error, genericKey, itemLabel) {
+     *  investigate the same way a network/permission error is.
+     *  `extraParams` carries the placeholders a few keys need on top of {{name}}/{{error}}
+     *  (e.g. errorFeatureAttachment's {{feature}}, errorPoi's {{sweep}}), which used to be
+     *  rendered as raw "{{feature}}" text in the log. */
+    logCopyItemError(error, genericKey, itemLabel, extraParams = {}) {
         if (error instanceof MissingS3FileError) {
             this.logItemError('errorMissingS3FileSkipped', {
+                ...extraParams,
                 name: itemLabel,
                 file: error.sourceKey
             });
         }
         else {
-            this.logItemError(genericKey, { name: itemLabel, error: toErrorMessage(error) });
+            this.logItemError(genericKey, { ...extraParams, name: itemLabel, error: toErrorMessage(error) });
         }
     }
-    //
-    // ---------- Duplicate detection ----------
-    //
     /**
-     * Heuristic duplicate-detection: compares title AND description together instead of
-     * title alone, to reduce false positives when several source items happen to share
-     * the same title. Not a perfect guarantee, but meaningfully more robust.
+     * Null-safe equivalent of FeatureService.isArchived(): true when the operator has archived
+     * this feature on that space. "No mission on the space" answers false (nothing archived)
+     * instead of throwing, which is what made a copy from a space the operator is not a member
+     * of fail outright.
      */
-    matchesExisting(source, targetList) {
-        return targetList.some(t => t.title === source.title && (t.description || null) === (source.description || null));
+    isArchivedForOperator(feature, spaceId) {
+        const archivedFor = feature?.archivedForMissionID;
+        if (!archivedFor || archivedFor.length === 0) {
+            return false;
+        }
+        const mission = this.userService.currentMission(spaceId);
+        return mission?.id ? archivedFor.includes(mission.id) : false;
+    }
+    //
+    // ---------- Cross-space reference remapping ----------
+    //
+    /** Normalised lookup key for a domain: type disambiguates same-named domains used for
+     *  different purposes (equipment vs. ticket vs. feature categories, ...). */
+    domainKey(type, name) {
+        return `${type || ''}::${(name || '').trim().toLowerCase()}`;
+    }
+    /**
+     * READ-ONLY indexing of every domain either operator's Mission can see. Creates
+     * nothing — unlike hashtags, a domain is a business taxonomy entry an operator manages
+     * deliberately; this tool only ever reuses an EXISTING equivalent, it never invents one.
+     *
+     * Visibility is keyed to `currentMission(spaceId).organisationID` /
+     * `.orderedOrganisationID` for EACH space — exactly what
+     * `DomainService.getDomainsForOrgForType()` uses to populate the category dropdown on
+     * the equipment/ticket/feature edit forms (see form-equipment.component.ts,
+     * batch-edit-equipment.component.ts, form-feature.component.ts). Comparing the spaces'
+     * OWN `organisationID` instead — the previous approach — missed every domain reachable
+     * only through the operator's `orderedOrganisationID` (set when a mission is created "on
+     * behalf of" another organisation — see form-mission.component.ts), which is exactly how
+     * a domain ends up visible on two spaces that belong to different organisations.
+     */
+    async loadDomainIndex(sourceSpaceId, targetSpaceId) {
+        const sourceMission = this.userService.currentMission(sourceSpaceId);
+        const targetMission = this.userService.currentMission(targetSpaceId);
+        const sourceOrgIds = [sourceMission?.organisationID, sourceMission?.orderedOrganisationID]
+            .filter((id) => !!id);
+        const targetOrgIds = [targetMission?.organisationID, targetMission?.orderedOrganisationID]
+            .filter((id) => !!id);
+        this.targetVisibleDomainOrgIds = new Set(targetOrgIds);
+        const allOrgIds = Array.from(new Set([...sourceOrgIds, ...targetOrgIds]));
+        if (allOrgIds.length === 0) {
+            // The operator has no mission on either space — nothing to resolve against;
+            // every domainID will be dropped, same as an unresolvable lookup elsewhere.
+            return;
+        }
+        try {
+            const domainLists = await Promise.all(allOrgIds.map(orgId => this.domainService.getAllGenericDomainsAndForOrganisation(orgId)));
+            for (const list of domainLists) {
+                for (const domain of list || []) {
+                    if (domain?.id) {
+                        this.domainsById.set(domain.id, domain);
+                    }
+                }
+            }
+        }
+        catch (error) {
+            console.warn('[MatterportTagCopyService] Could not read the domains, cross-organisation categories will be dropped:', error);
+            return;
+        }
+        for (const domain of this.domainsById.values()) {
+            const visibleToTarget = !domain.organisationID || this.targetVisibleDomainOrgIds.has(domain.organisationID);
+            if (visibleToTarget && domain.name) {
+                this.targetDomainIdsByTypeAndName.set(this.domainKey(domain.type, domain.name), domain.id);
+            }
+        }
+    }
+    /**
+     * Resolves a source domainID (equipment/ticket/feature/POI category) for use in the
+     * target space.
+     *
+     *  - generic domain (no organisationID of its own — a shared taxonomy entry, e.g. a
+     *    standard category created by the platform): reused unchanged, valid everywhere.
+     *  - the domain's own organisationID is already one the TARGET operator's mission can
+     *    see (its organisationID, or its orderedOrganisationID): reused unchanged too — the
+     *    id is directly selectable in the target space's own dropdown as-is.
+     *  - otherwise: matched by (type, name) against the domains visible to the TARGET
+     *    operator (loadDomainIndex()); dropped when no equivalent exists there.
+     */
+    resolveDomainID(sourceDomainId) {
+        if (!sourceDomainId) {
+            return null;
+        }
+        const domain = this.domainsById.get(sourceDomainId);
+        if (!domain) {
+            // Not visible to either operator's mission (lookup failed, or it belongs to
+            // neither) — unchanged, safest to drop.
+            return null;
+        }
+        if (!domain.organisationID || this.targetVisibleDomainOrgIds.has(domain.organisationID)) {
+            return sourceDomainId;
+        }
+        return this.targetDomainIdsByTypeAndName.get(this.domainKey(domain.type, domain.name)) ?? null;
+    }
+    /** Normalised lookup key for a hashtag name, so "Moyen Âge" and "moyen âge " match. */
+    hashtagKey(name) {
+        return name.trim().toLowerCase();
+    }
+    /**
+     * READ-ONLY indexing of both spaces' hashtags. Creates nothing.
+     *
+     * `Feature.hashtags` stores Hashtag *ids*, and a Hashtag is a per-space record, so the
+     * ids have to be translated by name. The translation table is built here; the missing
+     * keywords are created later, one at a time, only when a POI that actually uses them is
+     * about to be written (see resolveHashtagIds()).
+     */
+    async loadHashtagIndex(sourceSpaceId, targetSpaceId) {
+        let sourceHashtags = [];
+        let targetHashtags = [];
+        try {
+            [sourceHashtags, targetHashtags] = await Promise.all([
+                this.hashtagService.listHashtagsForSpace(sourceSpaceId),
+                this.hashtagService.listHashtagsForSpace(targetSpaceId)
+            ]);
+        }
+        catch (error) {
+            console.warn('[MatterportTagCopyService] Could not read the hashtags, keywords will be dropped:', error);
+            return;
+        }
+        for (const hashtag of sourceHashtags || []) {
+            if (hashtag?.id && hashtag.name) {
+                this.sourceHashtagNames.set(hashtag.id, hashtag.name);
+            }
+        }
+        for (const hashtag of targetHashtags || []) {
+            if (hashtag?.id && hashtag.name) {
+                this.targetHashtagIdsByName.set(this.hashtagKey(hashtag.name), hashtag.id);
+            }
+        }
+    }
+    /**
+     * Translates one feature's hashtag ids into target-space ids, creating on the fly the
+     * keywords the target space does not have yet.
+     *
+     * Creation is LAZY on purpose. Doing it up front, for the whole source vocabulary, meant
+     * that a copy which failed or was closed halfway left behind keywords no POI referenced —
+     * pollution the operator had to clean up by hand. Here a keyword is only ever written
+     * when a POI that carries it is about to be created, so an interrupted copy stops
+     * creating keywords at the same POI it stops creating content.
+     *
+     * The residual window (keyword created, then createFeature fails) is closed by
+     * markHashtagsUsed() + cleanupUnusedCreatedHashtags().
+     *
+     * A keyword that can neither be found nor created is dropped rather than left as a
+     * dangling id.
+     */
+    async resolveHashtagIds(sourceHashtagIds, targetSpaceId) {
+        if (!sourceHashtagIds || sourceHashtagIds.length === 0) {
+            return null;
+        }
+        const resolved = [];
+        for (const sourceId of sourceHashtagIds) {
+            if (!sourceId) {
+                continue;
+            }
+            const name = this.sourceHashtagNames.get(sourceId);
+            if (!name) {
+                // the source id points at a hashtag that no longer exists — nothing to carry over
+                continue;
+            }
+            const key = this.hashtagKey(name);
+            let targetId = this.targetHashtagIdsByName.get(key);
+            if (!targetId) {
+                try {
+                    const created = await this.hashtagService.createHashtagForSpace(name, targetSpaceId);
+                    targetId = created?.id;
+                    if (targetId) {
+                        this.targetHashtagIdsByName.set(key, targetId);
+                        this.createdHashtagIds.add(targetId);
+                    }
+                }
+                catch (error) {
+                    console.warn(`[MatterportTagCopyService] Could not create the hashtag "${name}" in the target space:`, error);
+                    continue;
+                }
+            }
+            if (targetId && !resolved.includes(targetId)) {
+                resolved.push(targetId);
+            }
+        }
+        return resolved.length > 0 ? resolved : null;
+    }
+    /**
+     * READ-ONLY indexing of both spaces' missions.
+     *
+     * A Mission is scoped to one space (it pairs a userID with a spaceID), so the id a
+     * source comment's `ownerID` points at never designates anyone in the target space.
+     * This builds the two lookup tables resolveCommentOwnerID() needs to translate it:
+     * which user a source Mission id belongs to, and which Mission (if any) that same
+     * user already holds in the target space.
+     */
+    async loadMissionIndex(sourceSpaceId, targetSpaceId) {
+        let sourceMissions = [];
+        let targetMissions = [];
+        try {
+            [sourceMissions, targetMissions] = await Promise.all([
+                this.missionService.getAllMissionsForSpace(sourceSpaceId),
+                this.missionService.getAllMissionsForSpace(targetSpaceId)
+            ]);
+        }
+        catch (error) {
+            console.warn('[MatterportTagCopyService] Could not read the missions, comment authorship will be dropped:', error);
+            return;
+        }
+        for (const mission of sourceMissions || []) {
+            if (mission?.id && mission.userID) {
+                this.sourceMissionUserIds.set(mission.id, mission.userID);
+            }
+        }
+        for (const mission of targetMissions || []) {
+            if (mission?.id && mission.userID && !this.targetMissionIdsByUserId.has(mission.userID)) {
+                this.targetMissionIdsByUserId.set(mission.userID, mission.id);
+            }
+        }
+    }
+    /**
+     * Translates a source comment's `ownerID` (a per-space Mission id) into the Mission of
+     * the SAME user in the target space, so the copy keeps crediting its real author.
+     *
+     * Substituting the current operator's mission — the way `ownerMissionID` legitimately
+     * does for tickets/equipment/features, which really do change owner on a copy — would
+     * misattribute every comment to whoever ran the copy instead of who actually wrote it.
+     *
+     * When the original author has no mission in the target space (never granted access
+     * there), the comment is left with no owner rather than a wrong one: an absent author
+     * is honest, a misattributed one is not.
+     */
+    resolveCommentOwnerID(sourceOwnerID) {
+        if (!sourceOwnerID) {
+            return null;
+        }
+        const userID = this.sourceMissionUserIds.get(sourceOwnerID);
+        if (!userID) {
+            return null;
+        }
+        return this.targetMissionIdsByUserId.get(userID) ?? null;
+    }
+    /** Records that these hashtags are now referenced by a feature that really exists, so
+     *  the end-of-run cleanup leaves them alone. */
+    markHashtagsUsed(hashtagIds) {
+        for (const id of hashtagIds || []) {
+            this.usedHashtagIds.add(id);
+        }
+    }
+    /**
+     * Deletes the hashtags this run created that no copied POI ended up referencing.
+     *
+     * Runs on every exit path — success, general failure, or exception — so the target space
+     * never keeps keywords that came from an aborted copy. Only ids created by THIS run are
+     * ever considered: a pre-existing keyword of the target space is never touched, even if
+     * no copied POI uses it.
+     */
+    async cleanupUnusedCreatedHashtags() {
+        const orphans = [...this.createdHashtagIds].filter(id => !this.usedHashtagIds.has(id));
+        if (orphans.length === 0) {
+            return;
+        }
+        console.log(`[MatterportTagCopyService] Removing ${orphans.length} keyword(s) created but left unused by this copy.`);
+        for (const hashtagId of orphans) {
+            try {
+                await this.hashtagService.deleteHashtagById(hashtagId);
+                this.createdHashtagIds.delete(hashtagId);
+            }
+            catch (error) {
+                // Cleanup is best-effort: a leftover keyword is harmless next to a failed copy,
+                // and must never be the reason the whole operation reports a failure.
+                console.warn(`[MatterportTagCopyService] Could not remove the unused keyword "${hashtagId}":`, error);
+            }
+        }
+    }
+    //
+    // ---------- Media content (Comments) ----------
+    //
+    /**
+     * Orders a feature's media set exactly the way the UI will read it back, and repairs
+     * the ordering key when the source data cannot express the user's intended order.
+     *
+     * The viewer sorts each media family on its own (`setImages`, `setVideos`, `setAudio`,
+     * `setDocuments`, ... all do `.sort((a, b) => a.index - b.index)`), so `index` only has
+     * to be consistent WITHIN a CommentType. The attachment form however defaults `index`
+     * to 1 for every new attachment, so a multi-image POI very often holds several images
+     * all sharing index 1 — or null. Sorting those is left to DynamoDB's arbitrary scan
+     * order, which is why the copy came out in a different order than the original.
+     *
+     * Strategy: sort deterministically (index, then creation date, then id — i.e. the
+     * upload order the user actually saw), and only rewrite the indices of a family when
+     * they are ambiguous (null or duplicated). A family whose indices are already unique
+     * is copied untouched, so a deliberate ordering is never overwritten.
+     */
+    orderCommentsForCopy(comments) {
+        const valid = (comments || []).filter(c => !!c);
+        const byType = new Map();
+        for (const comment of valid) {
+            const key = comment.type || 'UNTYPED';
+            if (!byType.has(key)) {
+                byType.set(key, []);
+            }
+            byType.get(key).push(comment);
+        }
+        const ordered = [];
+        for (const family of byType.values()) {
+            family.sort((a, b) => {
+                const ai = this.orderingIndexOf(a);
+                const bi = this.orderingIndexOf(b);
+                if (ai !== bi) {
+                    return ai - bi;
+                }
+                const ad = this.creationTimeOf(a);
+                const bd = this.creationTimeOf(b);
+                if (ad !== bd) {
+                    return ad - bd;
+                }
+                return String(a.id || '').localeCompare(String(b.id || ''));
+            });
+            const indices = family.map(c => c.index);
+            const isAmbiguous = indices.some(i => i === null || i === undefined)
+                || new Set(indices).size !== indices.length;
+            family.forEach((comment, position) => {
+                comment.__copyIndex = isAmbiguous ? position + 1 : comment.index;
+            });
+            ordered.push(...family);
+        }
+        return ordered;
+    }
+    /** Sort key for a comment's position; a missing index sorts last, not first. */
+    orderingIndexOf(comment) {
+        const index = comment?.index;
+        return index === null || index === undefined || Number.isNaN(Number(index))
+            ? Number.MAX_SAFE_INTEGER
+            : Number(index);
+    }
+    /** Creation timestamp of a comment, used as the tie-breaker: it reflects the order in
+     *  which the user uploaded the media, which is the order they saw. */
+    creationTimeOf(comment) {
+        const createdAt = comment?.createdAt;
+        if (!createdAt) {
+            return 0;
+        }
+        const time = new Date(createdAt).getTime();
+        return Number.isNaN(time) ? 0 : time;
+    }
+    /**
+     * Copies ONE media item (image, video, audio, document, text, link, embed, iframe...)
+     * of a feature, duplicating its file in S3 first when it has one.
+     *
+     * Every column of the Comment model is carried over. `description` and `metadata` were
+     * previously dropped, which emptied the captions of images/videos, the body of TEXT
+     * items — where the whole content lives, since a TEXT comment has no file — and any
+     * per-media settings stored in metadata.
+     */
+    async copyCommentForFeature(sourceComment, targetFeatureId, targetSpaceId) {
+        return this.copyComment(sourceComment, targetSpaceId, { featureID: targetFeatureId });
+    }
+    /** Same, for an attachment carried by a ticket rather than by a feature. */
+    async copyCommentForTicket(sourceComment, targetTicketId, targetSpaceId) {
+        return this.copyComment(sourceComment, targetSpaceId, { ticketID: targetTicketId });
+    }
+    /** Shared body of the two above: `owner` carries whichever foreign key links the copy back
+     *  to its new parent, everything else is identical. */
+    async copyComment(sourceComment, targetSpaceId, owner) {
+        const rawComment = sourceComment;
+        let copiedAnnexe = null;
+        if (rawComment.annexe) {
+            copiedAnnexe = await this.copyS3KeyIfPresent(rawComment.annexe, `attachment "${rawComment.title}"`);
+        }
+        const commentInput = {
+            title: rawComment.title,
+            description: rawComment.description ?? null,
+            metadata: rawComment.metadata ?? null,
+            type: rawComment.type,
+            annexe: copiedAnnexe,
+            externalLink: rawComment.externalLink || null,
+            dimensions: rawComment.dimensions || null,
+            index: rawComment.__copyIndex ?? rawComment.index ?? null,
+            shownInTag: rawComment.shownInTag,
+            isCoverImage: rawComment.isCoverImage,
+            spaceID: targetSpaceId,
+            // Keeps crediting the ORIGINAL author when they also have a mission in the
+            // target space, and leaves the comment un-owned otherwise — see
+            // resolveCommentOwnerID(). Never the current operator's own mission: that
+            // would silently reassign authorship to whoever happens to run the copy.
+            ownerID: this.resolveCommentOwnerID(rawComment.ownerID),
+            ...owner
+        };
+        return this.commentService.createComment(commentInput);
+    }
+    //
+    // ---------- POI creation ----------
+    //
+    /**
+     * Creates the target-space copy of a source POI, with EVERY column of the POI model.
+     *
+     * Single entry point for all phases: the element phases (ticket, equipment, 3D object,
+     * feature, measurement) used to build their POI inline with only 5 fields — coordinate,
+     * type, elementID, metadata, sweep — which is precisely how museum POIs lost their
+     * pastille: `tagDescription` (the popup text), `tagIcon` (the custom icon) and
+     * `tagImageID` (the pastille's image) never left the source space.
+     */
+    async createCopiedPoi(sourcePoi, elementID, sourceNavPositions, targetNavPositions, transform) {
+        const raw = sourcePoi;
+        const targetSweepId = this.findClosestTargetSweepId(sourcePoi.matterportSweepID, sourceNavPositions, targetNavPositions, transform);
+        const adjustedCoordinate = this.getAdjustedCoordinates(sourcePoi.coordinate, transform);
+        const poiInput = {
+            coordinate: adjustedCoordinate === null || adjustedCoordinate === undefined
+                ? null
+                : JSON.stringify(adjustedCoordinate),
+            type: sourcePoi.type,
+            elementID: elementID ?? null,
+            matterportSweepID: targetSweepId,
+            // --- pastille (tag popup) ---
+            tagDescription: sourcePoi.tagDescription ?? null,
+            tagIcon: await this.duplicateTagIcon(sourcePoi.tagIcon),
+            tagImageID: this.resolveTagImageID(sourcePoi.tagImageID),
+            // --- rendering / visibility flags, carried over verbatim ---
+            specific: raw.specific ?? null,
+            private: raw.private ?? null,
+            global: raw.global ?? null,
+            level: raw.level ?? null,
+            role: raw.role ?? null,
+            category: raw.category ?? null,
+            domainID: this.resolveDomainID(raw.domainID),
+            // --- Matterport anchoring ---
+            metadata: this.getAdjustedPoiMetadata(sourcePoi.metadata, transform)
+        };
+        return this.poiService.createPOIForSpaceID(poiInput);
+    }
+    /**
+     * Adapts a POI's `metadata` (see MattertagData.getMetadataString(): { icon, normal, sid })
+     * to the target space.
+     *
+     *  - `sid` is the id of the Mattertag inside the SOURCE Matterport model. Carried over,
+     *    the viewer tries to re-create the target's tag with an id belonging to another
+     *    model ("already in use"), so the pastille fails to attach. It is dropped: the
+     *    viewer then allocates a fresh sid on first render.
+     *  - `normal` is the stem direction. It is a direction, so only the rotation applies
+     *    (never the translation); without it, a copied pastille points into the wall
+     *    whenever the two models are not aligned on the same axis.
+     *  - `icon` and anything else is kept as-is.
+     */
+    getAdjustedPoiMetadata(metadata, transform) {
+        if (!metadata) {
+            return metadata ?? null;
+        }
+        try {
+            const parsed = JSON.parse(metadata);
+            if (!parsed || typeof parsed !== 'object') {
+                return metadata;
+            }
+            if (parsed.normal
+                && typeof parsed.normal === 'object'
+                && typeof parsed.normal.x === 'number'
+                && typeof parsed.normal.y === 'number'
+                && typeof parsed.normal.z === 'number') {
+                parsed.normal = applyRigidTransformToDirection(parsed.normal, transform);
+            }
+            delete parsed.sid;
+            return JSON.stringify(parsed);
+        }
+        catch {
+            // Not JSON, or unexpected shape — leave untouched rather than guess.
+            return metadata;
+        }
+    }
+    /**
+     * Resolves a POI's `tagImageID` — which the schema documents as a "connection to
+     * comment.id", i.e. the id of the Comment holding the image shown in the pastille.
+     *
+     * It is NOT an S3 key, so the previous "does it contain a slash?" heuristic never
+     * matched and the value travelled unchanged, leaving every copied pastille pointing at
+     * a Comment of the source space. It is now translated through the comment id map built
+     * while duplicating the media, and dropped when it cannot be resolved (the pastille
+     * then falls back to the feature's cover image instead of a broken reference).
+     */
+    resolveTagImageID(tagImageID) {
+        if (!tagImageID) {
+            return null;
+        }
+        return this.commentIds.get(tagImageID) ?? null;
+    }
+    //
+    // ---------- Duplicate detection ----------
+    //
+    /**
+     * Heuristic duplicate-detection: compares several identifying fields together rather than
+     * a single label, to reduce false positives when items share one of them. Returns the
+     * matched target item (or undefined) rather than a boolean, so the caller can act on the
+     * existing record — heal its missing fields — instead of only skipping it.
+     *
+     * The fields are supplied by the caller because the models do not agree on them. The
+     * previous version hard-coded `title`/`description`, which do not exist on Equipment at
+     * all: both sides read `undefined`, every equipment therefore "matched" every other one,
+     * and a target space holding a single equipment silently caused the whole equipment phase
+     * to copy nothing.
+     */
+    findExisting(source, targetList, identity) {
+        const sourceKey = this.identityKey(identity(source));
+        return targetList.find(target => this.identityKey(identity(target)) === sourceKey);
+    }
+    /**
+     * Builds a patch of whichever fields in `fields` the existing target record is missing
+     * (`null`/`undefined`/empty array) but the source has, plus `domainID` when requested —
+     * domains are organisation-scoped everywhere in this file, so that one is only ever
+     * healed across the SAME organisation, exactly like at creation time.
+     *
+     * Never overwrites a field the target already carries a value for: a value entered by
+     * hand in the target space since an earlier, incomplete copy must not be clobbered by
+     * an older or different source value.
+     *
+     * Returns both the patch to send and the list of healed field names, so the caller can
+     * both perform the update and describe it in the copy log.
+     */
+    buildHealPatch(sourceObj, existingTarget, fields, includeDomain) {
+        const patch = {};
+        const healedFields = [];
+        const fillIfMissing = (key, sourceValue) => {
+            const currentValue = existingTarget[key];
+            const isMissing = currentValue === null || currentValue === undefined
+                || (Array.isArray(currentValue) && currentValue.length === 0);
+            if (isMissing && sourceValue !== null && sourceValue !== undefined) {
+                patch[key] = sourceValue;
+                healedFields.push(key);
+            }
+        };
+        for (const field of fields) {
+            fillIfMissing(field, sourceObj[field]);
+        }
+        if (includeDomain && (existingTarget.domainID === null || existingTarget.domainID === undefined)) {
+            const resolvedDomainId = this.resolveDomainID(sourceObj.domainID);
+            if (resolvedDomainId) {
+                patch.domainID = resolvedDomainId;
+                healedFields.push('domainID');
+            }
+        }
+        return { patch, healedFields };
+    }
+    /**
+     * Fills in whichever identity field a PRE-EXISTING target equipment is missing but the
+     * source has — see the call site in the EQUIPMENTS phase for why re-running the copy
+     * has to be able to repair a previously-copied equipment, not just skip it again.
+     *
+     * A failed update is left to bubble up to the caller's existing per-item try/catch, the
+     * same as a failed creation — it gets the same visible error line, rather than a silent
+     * console warning.
+     *
+     * Returns the list of fields that were actually healed (empty if nothing was missing).
+     */
+    async healEquipmentIdentity(sourceObj, existingTarget) {
+        const { patch, healedFields } = this.buildHealPatch(sourceObj, existingTarget, ['status', 'annotation', 'family', 'sku', 'manufacturer', 'installDate', 'warrantyEndDate', 'metadata', 'isCreatedByAI'], true);
+        if (!existingTarget.photo && sourceObj.photo) {
+            patch.photo = await this.copyS3KeyIfPresent(sourceObj.photo, `equipment photo "${sourceObj.name}"`);
+            healedFields.push('photo');
+        }
+        if (healedFields.length === 0) {
+            return healedFields;
+        }
+        patch.id = existingTarget.id;
+        await this.equipmentService.updateEquipment(patch);
+        return healedFields;
+    }
+    /**
+     * Same idea as healEquipmentIdentity(), for a pre-existing ticket. `title`/`description`
+     * are the pair findExisting() matched on to find this ticket in the first place, so they
+     * are never part of the heal — by construction, they already agree with the source.
+     */
+    async healTicketIdentity(sourceObj, existingTarget) {
+        const { patch, healedFields } = this.buildHealPatch(sourceObj, existingTarget, ['friendlyLabel', 'status', 'type', 'priority', 'level', 'metadata', 'read', 'projectID', 'problemCode'], true);
+        if (healedFields.length === 0) {
+            return healedFields;
+        }
+        patch.id = existingTarget.id;
+        await this.ticketService.updateTicket(patch);
+        return healedFields;
+    }
+    /** Same idea, for a pre-existing feature. Keywords, media and POIs are not healed here
+     *  for the same reason notes/POIs are not healed on equipment — see healEquipmentIdentity(). */
+    async healFeatureIdentity(sourceObj, existingTarget) {
+        const { patch, healedFields } = this.buildHealPatch(sourceObj, existingTarget, ['shortDescription', 'level', 'type', 'index', 'stripeProductID', 'stripePriceID', 'bookingPriceMetadata', 'locale'], true);
+        if (healedFields.length === 0) {
+            return healedFields;
+        }
+        patch.id = existingTarget.id;
+        await this.featureService.updateFeature(patch);
+        return healedFields;
+    }
+    /** Serialises the identifying fields unambiguously, so that ["a", "b"] and ["ab", ""]
+     *  can never produce the same key. */
+    identityKey(parts) {
+        return JSON.stringify(parts.map(part => part ?? ''));
     }
     //
     // ---------- Sweep positions / rigid transform ----------
@@ -14808,10 +16045,12 @@ class MatterportTagCopyService {
     /**
      * Builds a new S3 key for a copied attachment and sanitizes the filename
      * to prevent AWS Amplify SignatureDoesNotMatch errors.
+     * `folderOverride` re-homes the copy into another prefix — used for tag icons, which
+     * are stored per space (`icons-tags/<spaceId>/`) and must follow the target space.
      */
-    buildCopiedAttachmentKey(sourceAnnexe) {
+    buildCopiedAttachmentKey(sourceAnnexe, folderOverride = null) {
         const lastSlash = sourceAnnexe.lastIndexOf('/');
-        const folder = sourceAnnexe.substring(0, lastSlash + 1);
+        const folder = folderOverride ?? sourceAnnexe.substring(0, lastSlash + 1);
         let filename = sourceAnnexe.substring(lastSlash + 1);
         // Strip a leading "<8 hex chars>_" id prefix if present
         const prefixMatch = filename.match(/^[0-9a-fA-F-]{8}_(.+)$/);
@@ -14831,12 +16070,12 @@ class MatterportTagCopyService {
      *   skip the whole item entirely, rather than creating a record with a broken file link.
      * - Any other kind of failure (permissions, network, etc.) still throws a generic Error.
      */
-    async copyS3KeyIfPresent(key, contextLabel) {
+    async copyS3KeyIfPresent(key, contextLabel, folderOverride = null) {
         if (!key) {
             return null;
         }
         const sourceKey = this.extractS3KeyFromValue(key);
-        const destKey = this.buildCopiedAttachmentKey(sourceKey);
+        const destKey = this.buildCopiedAttachmentKey(sourceKey, folderOverride);
         // Log the source key to ensure it matches the real S3 object path
         console.log(`[DEBUG] Attempting S3 copy from source: "${sourceKey}" to dest: "${destKey}"`);
         try {
@@ -14847,49 +16086,101 @@ class MatterportTagCopyService {
             return copied;
         }
         catch (error) {
-            const isMissingFile = error instanceof MissingS3FileError
-                || error?.code === 'NoSuchKey'
-                || /NoSuchKey|does not exist/i.test(toErrorMessage(error));
-            if (isMissingFile) {
+            if (this.isMissingS3File(error)) {
                 console.warn(`[MatterportTagCopyService] Orphaned S3 reference, skipping item: "${sourceKey}" (${contextLabel})`);
                 throw new MissingS3FileError(sourceKey, contextLabel);
             }
+            // Storage.copy() cannot handle a key holding a space, an accent or a parenthesis:
+            // it puts the key unencoded into `x-amz-copy-source`, which then disagrees with the
+            // signed request. Those files are perfectly readable — only that header is broken —
+            // so fall back to downloading and re-uploading them.
+            if (this.isSignatureMismatch(error)) {
+                console.warn(`[MatterportTagCopyService] Storage.copy() signature mismatch on "${sourceKey}", falling back to download + re-upload.`);
+                try {
+                    const copied = await copyFileInS3ViaDownload(sourceKey, destKey);
+                    if (!copied) {
+                        throw new MissingS3FileError(sourceKey, contextLabel);
+                    }
+                    return copied;
+                }
+                catch (fallbackError) {
+                    if (this.isMissingS3File(fallbackError)) {
+                        throw new MissingS3FileError(sourceKey, contextLabel);
+                    }
+                    throw fallbackError;
+                }
+            }
             throw error;
         }
     }
+    /** True when the failure means the source object simply is not on S3 (orphaned reference). */
+    isMissingS3File(error) {
+        return error instanceof MissingS3FileError
+            || error?.code === 'NoSuchKey'
+            || error?.name === 'NoSuchKey'
+            || /NoSuchKey|does not exist|NotFound/i.test(toErrorMessage(error));
+    }
+    /** True when AppSync refused the operation for lack of permission, as opposed to failing it.
+     *  Nothing can be retried or worked around client-side: the account simply is not allowed
+     *  to write that model, which is a backend authorisation rule (@auth) to change. */
+    isUnauthorized(error) {
+        const message = toErrorMessage(error);
+        return error?.errorType === 'Unauthorized'
+            || /Unauthorized|Not Authorized to access/i.test(message);
+    }
+    /** True for the SigV4 mismatch Storage.copy() raises on keys with special characters. */
+    isSignatureMismatch(error) {
+        return error?.code === 'SignatureDoesNotMatch'
+            || error?.name === 'SignatureDoesNotMatch'
+            || /signature we calculated does not match|SignatureDoesNotMatch/i.test(toErrorMessage(error));
+    }
     /**
      * Duplicates the file referenced by a POI's tagIcon (a JSON string shaped like
-     * { src, opacity }, per form-feature.component.ts). If the JSON can't be parsed or
-     * has no `src`, the value is returned unchanged rather than risk corrupting it.
+     * { src, opacity }, per form-feature.component.ts).
+     *
+     * Two behaviours matter here:
+     *  - the copy is re-homed under the TARGET space's icon folder
+     *    (`icons-tags/<targetSpaceId>/`, the layout form-feature.component.ts uploads to).
+     *    Keeping the source folder produced a target space whose icons all lived under
+     *    another space's prefix, so cleaning up the source space broke the copy's icons.
+     *  - a missing icon file is NOT fatal. It used to bubble up as a MissingS3FileError and
+     *    abort the whole POI — losing the pastille, its text and its media because of a
+     *    decorative file. The POI is now created with the icon reference dropped, which
+     *    renders the module's default icon, and a log line reports it.
      */
     async duplicateTagIcon(tagIconRaw) {
         if (!tagIconRaw) {
-            return tagIconRaw;
+            return tagIconRaw ?? null;
         }
+        let parsed;
         try {
-            const parsed = JSON.parse(tagIconRaw);
-            if (parsed && parsed.src) {
-                parsed.src = await this.copyS3KeyIfPresent(parsed.src, 'tag icon');
-            }
-            return JSON.stringify(parsed);
+            parsed = JSON.parse(tagIconRaw);
         }
         catch {
-            // Not JSON, or unexpected shape — leave untouched rather than guess.
+            // Not JSON — leave untouched rather than guess.
             return tagIconRaw;
         }
-    }
-    /**
-     * Duplicates tagImageID if it looks like a reference to a file (contains a path
-     * separator, whether raw key or pre-signed URL). UNCONFIRMED ASSUMPTION: this
-     * field's exact semantics weren't verified — if it's actually an opaque reference
-     * to another DB record rather than a file, this heuristic may incorrectly attempt
-     * a copy. Confirm the real structure to tighten this if it causes issues.
-     */
-    async duplicateTagImageID(tagImageID) {
-        if (!tagImageID || !tagImageID.includes('/')) {
-            return tagImageID;
+        if (!parsed || typeof parsed !== 'object' || !parsed.src) {
+            return tagIconRaw;
+        }
+        try {
+            parsed.src = await this.copyS3KeyIfPresent(parsed.src, 'tag icon', this.buildTargetIconFolder());
+        }
+        catch (iconError) {
+            if (!(iconError instanceof MissingS3FileError)) {
+                // A real failure (permissions, network, ...) still aborts the POI: it means
+                // the copy cannot be trusted, unlike a file that is simply already gone.
+                throw iconError;
+            }
+            console.warn(`[MatterportTagCopyService] Tag icon missing on S3, falling back to the default icon: "${iconError.sourceKey}"`);
+            this.logItemError('warningTagIconMissing', { file: iconError.sourceKey });
+            delete parsed.src;
         }
-        return this.copyS3KeyIfPresent(tagImageID, 'tag image');
+        return JSON.stringify(parsed);
+    }
+    /** Icon folder of the space being copied into, matching what the feature form uploads to. */
+    buildTargetIconFolder() {
+        return this.targetSpaceId ? `icons-tags/${this.targetSpaceId}/` : null;
     }
     /**
      * Extracts and normalizes the raw S3 key from a stored value (which may be a full URL
@@ -14932,13 +16223,13 @@ class MatterportTagCopyService {
         // for the CopySource header. Do NOT re-encode it here.
         return extracted;
     }
-    static ɵfac = i0.ɵɵngDeclareFactory({ minVersion: "12.0.0", version: "17.3.12", ngImport: i0, type: MatterportTagCopyService, deps: [{ token: NavigationService }, { token: BaseUserService }, { token: PoiService }, { token: TicketsService }, { token: EquipmentService }, { token: Object3DService }, { token: FeatureService }, { token: MeasurementService }, { token: CommentService }, { token: i1.TranslateService }], target: i0.ɵɵFactoryTarget.Injectable });
+    static ɵfac = i0.ɵɵngDeclareFactory({ minVersion: "12.0.0", version: "17.3.12", ngImport: i0, type: MatterportTagCopyService, deps: [{ token: NavigationService }, { token: BaseUserService }, { token: PoiService }, { token: TicketsService }, { token: EquipmentService }, { token: Object3DService }, { token: FeatureService }, { token: MeasurementService }, { token: CommentService }, { token: HashtagService }, { token: MissionService }, { token: OperationService }, { token: DomainService }, { token: i1.TranslateService }], target: i0.ɵɵFactoryTarget.Injectable });
     static ɵprov = i0.ɵɵngDeclareInjectable({ minVersion: "12.0.0", version: "17.3.12", ngImport: i0, type: MatterportTagCopyService, providedIn: 'root' });
 }
 i0.ɵɵngDeclareClassMetadata({ minVersion: "12.0.0", version: "17.3.12", ngImport: i0, type: MatterportTagCopyService, decorators: [{
             type: Injectable,
             args: [{ providedIn: 'root' }]
-        }], ctorParameters: () => [{ type: NavigationService }, { type: BaseUserService }, { type: PoiService }, { type: TicketsService }, { type: EquipmentService }, { type: Object3DService }, { type: FeatureService }, { type: MeasurementService }, { type: CommentService }, { type: i1.TranslateService }] });
+        }], ctorParameters: () => [{ type: NavigationService }, { type: BaseUserService }, { type: PoiService }, { type: TicketsService }, { type: EquipmentService }, { type: Object3DService }, { type: FeatureService }, { type: MeasurementService }, { type: CommentService }, { type: HashtagService }, { type: MissionService }, { type: OperationService }, { type: DomainService }, { type: i1.TranslateService }] });
 
 /* eslint-disable class-methods-use-this */
 class OrganisationService {
@@ -15012,81 +16303,6 @@ i0.ɵɵngDeclareClassMetadata({ minVersion: "12.0.0", version: "17.3.12", ngImpo
                     args: ['currentAPIService']
                 }] }] });
 
-class DomainService {
-    API; //AWS
-    constructor(apiInjected) {
-        this.API = apiInjected;
-    }
-    async getGenericDomainsAndForOrganisation(organisationID) {
-        return this.API.__proto__.ListDomains().then((domains) => {
-            return domains.items.filter((dom) => {
-                return (dom.organisationID === organisationID || !dom.organisationID);
-            });
-        });
-    }
-    async getGenericDomainsAndForOrgAndOrderedOrg(orgID, orderedOrgID = null) {
-        // TODO: Change this (UUID can change depending on DB)
-        if (orgID === "6e28201f-4679-4c29-9d2d-8a325f57cee9") {
-            // I am SmarterPlan, i can see all
-            return this.API.__proto__.ListDomains().then((response) => response.items);
-        }
-        const currentOrgDomains = await this.getGenericDomainsAndForOrganisation(orgID);
-        if (!orderedOrgID || orgID === orderedOrgID) {
-            return currentOrgDomains;
-        }
-        const orderedOrgDomains = await this.getGenericDomainsAndForOrganisation(orderedOrgID);
-        const merged = [...currentOrgDomains, ...orderedOrgDomains];
-        return filterUniqueArrayByID(merged);
-    }
-    async getDomainsForOrgExcludingTypes(types, organisationID) {
-        return this.getGenericDomainsAndForOrganisation(organisationID).then((domains) => {
-            return domains.filter((dom) => !types.includes(dom.type));
-        });
-    }
-    async getDomain(id) {
-        return this.API.__proto__.GetDomain(id);
-    }
-    async updateDomain(input) {
-        return this.API.__proto__.UpdateDomain(input);
-    }
-    async deleteDomain(id) {
-        return this.API.__proto__.DeleteDomain({ id });
-    }
-    async getRootDomains(orgID) {
-        const allDomains = await this.getGenericDomainsAndForOrganisation(orgID);
-        const root = allDomains.filter((dom) => !dom.parentID);
-        return root.sort((a, b) => a.name.localeCompare(b.name));
-    }
-    async createDomain(input) {
-        return this.API.__proto__.CreateDomain(input);
-    }
-    async getDomainsForOrgForType(type, organisationID, orderedOrg = null) {
-        return this.getGenericDomainsAndForOrgAndOrderedOrg(organisationID, orderedOrg).then((domains) => {
-            return domains.filter((dom) => dom.type === type);
-        });
-    }
-    async getParentDomainsForOrgAndOrderedOrgForType(type, organisationID, orderedOrg = null) {
-        return this.getGenericDomainsAndForOrgAndOrderedOrg(organisationID, orderedOrg).then((domains) => {
-            return domains.filter((dom) => dom.type === type && !dom.parentID);
-        });
-    }
-    async getDomainsForMissionsForOrg(orgID) {
-        const domains = await this.getDomainsForOrgExcludingTypes([DomainType.BOOKING, DomainType.FEATURE, DomainType.MISSION], orgID);
-        return domains.filter((dom) => !dom.parentID);
-    }
-    static ɵfac = i0.ɵɵngDeclareFactory({ minVersion: "12.0.0", version: "17.3.12", ngImport: i0, type: DomainService, deps: [{ token: 'currentAPIService' }], target: i0.ɵɵFactoryTarget.Injectable });
-    static ɵprov = i0.ɵɵngDeclareInjectable({ minVersion: "12.0.0", version: "17.3.12", ngImport: i0, type: DomainService, providedIn: "root" });
-}
-i0.ɵɵngDeclareClassMetadata({ minVersion: "12.0.0", version: "17.3.12", ngImport: i0, type: DomainService, decorators: [{
-            type: Injectable,
-            args: [{
-                    providedIn: "root",
-                }]
-        }], ctorParameters: () => [{ type: undefined, decorators: [{
-                    type: Inject,
-                    args: ['currentAPIService']
-                }] }] });
-
 class TemplateService {
     API; //AWS
     constructor(apiInjected) {
@@ -15147,5 +16363,5 @@ function floatValidator() {
  * Generated bundle index. Do not edit.
  */
 
-export { AffectationService, AvatarComponent, BaseLateralTabService, BaseTagService, BaseUserService, BaseVisibilityService, CameraMode, CaptureService, CaptureViewer, CommentService, CommentType, Config, ContentService, CsvExportComponent, DomainService, DomainType, DurationToStringPipe, EmailStatus, EquipmentService, EventService, EventStatus, EventType, FeatureService, FeatureType, FilterService, HashtagFromIdPipe, HashtagService, InterventionService, InventoryStatus, LayerService, LevelStatus, LoaderComponent, Locale, LocaleService, MATTERPORT_SDK_KEY, MatterportImportService, MatterportService, MatterportTagCopyService, MatterportViewType, MattertagActionMode, MattertagData, MeasurementService, MenuBarComponent, MissionService, NavigationService, NavigatorService, NgxSmarterplanCoreModule, NgxSmarterplanCoreService, NodeService, Object3DService, OperationService, OrganisationService, PaymentStatus, PlanService, PoiService, PoiType, ProfileEntity, ProfileService, ProfileStatus, PropertyService, PropertyType, RoleStatus, SafeUrlPipe, SearchBarComponent, SearchObjectType, SearchService, SpModule, SpaceService, SpaceStatus, StatusEquipment, SupportModalComponent, SupportService, TagAction, TemplateService, TicketPriority, TicketStatus, TicketType, TicketsService, TimeDateToLocalStringPipe, TypeNote, UsernameFromIdPipe, ValidatorsService, ViewerInteractions, ViewerService, VisitService, ZoneChangeService, ZoneService, applyRigidTransformToPoint, applyRigidTransformToYRotationDeg, arraysContainSameElements, checkElementById, computeRigidTransform, convertTimestampToLocalZone, copyFileInS3, dateHasExpired, dateTimeToLocalString, deleteFromS3, distance, downloadBlob, downloadEquipmentDocument, downloadFile, downloadFileAsObject, durationToString, emailValidator, enumToArray, filterUniqueArrayByID, floatValidator, getBufferForFileFromS3, getCoefficientsForImage, getCurrentLang, getDistanceBetweenTwoPoints, getHighestLevelForMissions, getHighestRoleForMissions, getLevelsBelow, getLocaleLong, getLocaleShort, getMetaForImage, getRolesBelowForManager, getSignedFile, getSignedImageUrlForEquipment, getSignedImageUrlForProfile, getSignedImageUrlForSpace, getSpaceIDFromUrl, getVisitUrl, isEmptyObject, listFilesInFolder, mean, noEmptyValidator, numberToDateString, openDocument, openModalForVisitSwitch, poiTypeToString, pointToString, removeAllFilesFromFolderS3, removeNullKeysFromObject, showScanPointsOnPlanInDiv, shuffleArray, sortAlphabeticallyOnName, stringLocaleToEnum, stringToLowercaseNoSpaces, styleButton, textValidator, translateDatePeriod, uploadBase64Image, uploadBase64ImageWithRetry, uploadFileToS3, uploadJsonToS3, validEmail, wait, waitUntil };
+export { AffectationService, AvatarComponent, BaseLateralTabService, BaseTagService, BaseUserService, BaseVisibilityService, CameraMode, CaptureService, CaptureViewer, CommentService, CommentType, Config, ContentService, CsvExportComponent, DomainService, DomainType, DurationToStringPipe, EmailStatus, EquipmentService, EventService, EventStatus, EventType, FeatureService, FeatureType, FilterService, HashtagFromIdPipe, HashtagService, InterventionService, InventoryStatus, LayerService, LevelStatus, LoaderComponent, Locale, LocaleService, MATTERPORT_SDK_KEY, MatterportImportService, MatterportService, MatterportTagCopyService, MatterportViewType, MattertagActionMode, MattertagData, MeasurementService, MenuBarComponent, MissionService, NavigationService, NavigatorService, NgxSmarterplanCoreModule, NgxSmarterplanCoreService, NodeService, Object3DService, OperationService, OrganisationService, PaymentStatus, PlanService, PoiService, PoiType, ProfileEntity, ProfileService, ProfileStatus, PropertyService, PropertyType, RoleStatus, SafeUrlPipe, SearchBarComponent, SearchObjectType, SearchService, SpModule, SpaceService, SpaceStatus, StatusEquipment, SupportModalComponent, SupportService, TagAction, TemplateService, TicketPriority, TicketStatus, TicketType, TicketsService, TimeDateToLocalStringPipe, TypeNote, UsernameFromIdPipe, ValidatorsService, ViewerInteractions, ViewerService, VisitService, ZoneChangeService, ZoneService, applyRigidTransformToDirection, applyRigidTransformToPoint, applyRigidTransformToYRotationDeg, arraysContainSameElements, checkElementById, computeRigidTransform, convertTimestampToLocalZone, copyFileInS3, copyFileInS3ViaDownload, dateHasExpired, dateTimeToLocalString, deleteFromS3, distance, downloadBlob, downloadEquipmentDocument, downloadFile, downloadFileAsObject, durationToString, emailValidator, enumToArray, filterUniqueArrayByID, floatValidator, getBufferForFileFromS3, getCoefficientsForImage, getCurrentLang, getDistanceBetweenTwoPoints, getHighestLevelForMissions, getHighestRoleForMissions, getLevelsBelow, getLocaleLong, getLocaleShort, getMetaForImage, getRolesBelowForManager, getSignedFile, getSignedImageUrlForEquipment, getSignedImageUrlForProfile, getSignedImageUrlForSpace, getSpaceIDFromUrl, getVisitUrl, isEmptyObject, listFilesInFolder, mean, noEmptyValidator, numberToDateString, openDocument, openModalForVisitSwitch, poiTypeToString, pointToString, removeAllFilesFromFolderS3, removeNullKeysFromObject, showScanPointsOnPlanInDiv, shuffleArray, sortAlphabeticallyOnName, stringLocaleToEnum, stringToLowercaseNoSpaces, styleButton, textValidator, translateDatePeriod, uploadBase64Image, uploadBase64ImageWithRetry, uploadFileToS3, uploadJsonToS3, validEmail, wait, waitUntil };
 //# sourceMappingURL=smarterplan-ngx-smarterplan-core.mjs.map