@agenticprimitives/contracts 1.0.0-alpha.5 → 1.0.0-alpha.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agenticprimitives/contracts",
-  "version": "1.0.0-alpha.5",
+  "version": "1.0.0-alpha.6",
   "description": "Solidity contracts + ABIs + flattened sources + per-network deployment addresses for the agenticprimitives stack.",
   "license": "MIT",
   "repository": {

package/src/geo/GeoFeatureRegistry.sol

@@ -121,29 +121,36 @@ contract GeoFeatureRegistry {
         }
 
         version = prevVer + 1;
-        _records[p.featureId][version] = FeatureRecord({
-            featureId: p.featureId,
-            version: version,
-            stewardAccount: p.stewardAccount,
-            featureKind: p.featureKind,
-            geometryHash: p.geometryHash,
-            coverageRoot: p.coverageRoot,
-            sourceSetRoot: p.sourceSetRoot,
-            metadataURI: p.metadataURI,
-            centroidLat: p.centroidLat,
-            centroidLon: p.centroidLon,
-            bboxMinLat: p.bboxMinLat,
-            bboxMinLon: p.bboxMinLon,
-            bboxMaxLat: p.bboxMaxLat,
-            bboxMaxLon: p.bboxMaxLon,
-            validAfter: p.validAfter,
-            validUntil: p.validUntil,
-            active: true,
-            registeredAt: uint64(block.timestamp)
-        });
         latestVersion[p.featureId] = version;
         if (prevVer == 0) _allFeatures.push(p.featureId);
+        // The heavy record write + event live in a helper so `publish`'s own stack stays shallow.
+        _storeAndEmit(p, version);
+    }
 
+    /// @dev Writes the record FIELD-BY-FIELD to storage (no in-memory struct literal) and emits.
+    ///      Behaviour is identical to the prior `_records[...] = FeatureRecord({...})` literal; the
+    ///      split exists only to keep `publish` within solc's stack under `forge coverage --ir-minimum`
+    ///      instrumentation (the 18-field memory literal + a dynamic `string` overflowed it by one slot).
+    function _storeAndEmit(PublishInput calldata p, uint64 version) internal {
+        FeatureRecord storage r = _records[p.featureId][version];
+        r.featureId = p.featureId;
+        r.version = version;
+        r.stewardAccount = p.stewardAccount;
+        r.featureKind = p.featureKind;
+        r.geometryHash = p.geometryHash;
+        r.coverageRoot = p.coverageRoot;
+        r.sourceSetRoot = p.sourceSetRoot;
+        r.metadataURI = p.metadataURI;
+        r.centroidLat = p.centroidLat;
+        r.centroidLon = p.centroidLon;
+        r.bboxMinLat = p.bboxMinLat;
+        r.bboxMinLon = p.bboxMinLon;
+        r.bboxMaxLat = p.bboxMaxLat;
+        r.bboxMaxLon = p.bboxMaxLon;
+        r.validAfter = p.validAfter;
+        r.validUntil = p.validUntil;
+        r.active = true;
+        r.registeredAt = uint64(block.timestamp);
         emit FeaturePublished(p.featureId, version, p.stewardAccount, p.featureKind, p.geometryHash, p.coverageRoot, p.sourceSetRoot, p.metadataURI);
     }
 

package/src/skills/SkillDefinitionRegistry.sol

@@ -116,24 +116,30 @@ contract SkillDefinitionRegistry {
 
         version = prevVer + 1;
         bytes32 predecessorRoot = prevVer == 0 ? bytes32(0) : _records[p.skillId][prevVer].ontologyMerkleRoot;
-
-        _records[p.skillId][version] = SkillRecord({
-            skillId: p.skillId,
-            version: version,
-            stewardAccount: p.stewardAccount,
-            skillKind: p.skillKind,
-            conceptHash: p.conceptHash,
-            ontologyMerkleRoot: p.ontologyMerkleRoot,
-            predecessorRoot: predecessorRoot,
-            metadataURI: p.metadataURI,
-            validAfter: p.validAfter,
-            validUntil: p.validUntil,
-            active: true,
-            registeredAt: uint64(block.timestamp)
-        });
         latestVersion[p.skillId] = version;
         if (prevVer == 0) _allSkills.push(p.skillId);
+        // The heavy record write + event live in a helper so `publish`'s own stack stays shallow.
+        _storeAndEmit(p, version, predecessorRoot);
+    }
 
+    /// @dev Writes the record FIELD-BY-FIELD to storage (no in-memory struct literal) and emits.
+    ///      Behaviour is identical to the prior `_records[...] = SkillRecord({...})` literal; the split
+    ///      exists only to keep `publish` within solc's stack under `forge coverage --ir-minimum`
+    ///      instrumentation (the memory struct literal + a dynamic `string` overflowed it by one slot).
+    function _storeAndEmit(PublishInput calldata p, uint64 version, bytes32 predecessorRoot) internal {
+        SkillRecord storage r = _records[p.skillId][version];
+        r.skillId = p.skillId;
+        r.version = version;
+        r.stewardAccount = p.stewardAccount;
+        r.skillKind = p.skillKind;
+        r.conceptHash = p.conceptHash;
+        r.ontologyMerkleRoot = p.ontologyMerkleRoot;
+        r.predecessorRoot = predecessorRoot;
+        r.metadataURI = p.metadataURI;
+        r.validAfter = p.validAfter;
+        r.validUntil = p.validUntil;
+        r.active = true;
+        r.registeredAt = uint64(block.timestamp);
         emit SkillPublished(p.skillId, version, p.stewardAccount, p.skillKind, p.conceptHash, p.ontologyMerkleRoot, predecessorRoot, p.metadataURI);
     }