Merge remote-tracking branch 'origin/development' into feat/snapshot-facet

# Conflicts:
#	packages/ats/contracts/contracts/domain/asset/nominalValue/NominalValueStorageWrapper.sol
#	packages/ats/contracts/scripts/domain/atsRegistry.data.ts

Signed-off-by: Mario Francia <mariofranciarius@gmail.com>

Author: themariofrancia

.changeset/maf-balance-tracker-adjusted-facet.md

@@ -0,0 +1,20 @@
+---
+"@hashgraph/asset-tokenization-contracts": minor
+---
+
+# BalanceTrackerAdjustedFacet split
+
+Extract `balanceOfAt` from `ERC1410ReadFacet` into a dedicated `BalanceTrackerAdjustedFacet` registered under `_BALANCE_TRACKER_ADJUSTED_RESOLVER_KEY`.
+
+## Changes
+
+- Added `contracts/facets/balanceTrackerAdjusted/IBalanceTrackerAdjusted.sol`, `BalanceTrackerAdjusted.sol`, `BalanceTrackerAdjustedFacet.sol`.
+- Added `_BALANCE_TRACKER_ADJUSTED_RESOLVER_KEY` constant in `resolverKeys.sol`.
+- Removed `balanceOfAt` from `ERC1410Read.sol`, `IERC1410Read.sol`, and `ERC1410ReadFacet.sol`; selector count drops from 9 to 8. Also migrated `ERC1410ReadFacet` selector registration to the `--selectorIndex`/`unchecked` pattern.
+- `IAsset` now exposes `balanceOfAt` via `IBalanceTrackerAdjusted`.
+- Updated `Configuration.ts`, `orchestratorLibraries.ts` and all 7 `createConfiguration.ts` scripts to include `BalanceTrackerAdjustedFacet`.
+- Added `test/contracts/integration/balanceTrackerAdjusted/balanceTrackerAdjusted.test.ts` covering: balance at current time, zero balance, timestamp 0, single scheduled adjustment before/after, and multiple chained adjustments.
+
+## Non-breaking
+
+The 4-byte selector of `balanceOfAt` is unchanged. Any call to `asset.balanceOfAt(...)` through `IAsset` continues to work without modification.

.changeset/maf-balance-tracker-by-partition-facet.md

@@ -0,0 +1,17 @@
+---
+"@hashgraph/asset-tokenization-contracts": major
+---
+
+feat(ats-contracts): add BalanceTrackerByPartitionFacet and remove TotalBalanceFacet
+
+Introduces `BalanceTrackerByPartitionFacet` with three partition-scoped read functions:
+
+- `balanceOfByPartition` (moved from `ERC1410ReadFacet`)
+- `totalSupplyByPartition` (moved from `ERC1410ReadFacet`)
+- `getTotalBalanceForByPartition` (moved from `TotalBalanceFacet`, which is removed)
+
+**Breaking changes:**
+
+- `IERC1410Read` no longer declares `balanceOfByPartition` or `totalSupplyByPartition`; these are now on `IBalanceTrackerByPartition`. This changes `IERC1410.interfaceId` (ERC-165).
+- `TotalBalanceFacet` and its interface `ITotalBalance` are deleted. Use `BalanceTrackerByPartitionFacet` instead.
+- All token configurations updated to replace `TotalBalanceFacet` with `BalanceTrackerByPartitionFacet`.

.changeset/maf-operator-clearing-hold-by-partition-facet.md

@@ -0,0 +1,5 @@
+---
+"@hashgraph/asset-tokenization-contracts": major
+---
+
+Refactor `operatorClearingCreateHoldByPartition` from `ClearingHoldCreation` contract into a new `OperatorClearingHoldByPartition` facet, registered under `_OPERATOR_CLEARING_HOLDBYPARTITION_RESOLVER_KEY`.

.changeset/maf-split-burn.md

@@ -0,0 +1,6 @@
+---
+"@hashgraph/asset-tokenization-contracts": major
+"@hashgraph/asset-tokenization-sdk": patch
+---
+
+Split a new Burn Facet. ERC3643OperationsFacet is empty so is deleted

.changeset/maf-split-controller-hold-by-partition.md

@@ -0,0 +1,5 @@
+---
+"@hashgraph/asset-tokenization-contracts": major
+---
+
+Refactor `controllerCreateHoldByPartition` function from `HoldManagement` contract into a new `controllerHoldByPartition` facet.

.claude/skills/solidity-natspec/SKILL.md

@@ -0,0 +1,93 @@
+---
+name: solidity-natspec
+description: Produce and validate comprehensive NatSpec documentation on Solidity files. Use whenever a `.sol` file is created or modified — contracts, interfaces, libraries, facets under `packages/ats/contracts/**`. Ensures every contract element (contract/interface/library, enums, structs, events, custom errors, state variables, modifiers, functions) carries audit-ready NatSpec in the project's house style (British English, ≤100-char lines, intent-focused). Also use on explicit request (`/solidity-natspec`) to document or re-document an existing `.sol` file.
+---
+
+# Solidity NatSpec Documentation
+
+Produce audit-ready NatSpec on every element the edit touches. Document **intent, behaviour,
+constraints, and structure** — never restate what the code already says. Audience: auditors,
+maintainers, analysis tools.
+
+When editing an existing file, document new or modified elements only. On `/solidity-natspec [path]`,
+do a full pass on the file.
+
+## Templates
+
+Place each block immediately above the element. Declaration order inside a contract:
+enums → structs → events → custom errors → state vars → modifiers → functions
+(`external`/`public` first, then `internal`/`private`).
+
+For contract / interface / library headers, `@title`, `@author` and `@notice` are **mandatory** —
+solhint's `use-natspec` rule flags each missing tag. Use
+`@author Asset Tokenization Studio Team` unless the file is clearly a fork of upstream code
+(OpenZeppelin, ERC references).
+
+```solidity
+/**
+ * @title <name>
+ * @author Asset Tokenization Studio Team
+ * @notice <high-level purpose>
+ * @dev <design notes, patterns, assumptions, invariants>
+ */
+```
+
+For every other element, include `@notice` (what and why), `@dev` when non-obvious, and one
+`@param`/`@return` per named parameter/return with matching names. Functions, events, and errors
+that omit `@notice` or mismatch `@param`/`@return` names fail solhint.
+
+For interface implementations, use `@inheritdoc IFoo` on the concrete function (plus a `@dev` only
+if the implementation introduces behaviour the interface doesn't describe — e.g. a pause gate, a
+facet-specific guard, a snapshot side-effect). Don't duplicate the interface block; it drifts.
+
+## Style
+
+- **British English**: _decentralised_, _behaviour_, _initialised_, _optimise_, _authorised_,
+  _organisation_, _serialise_, _analyse_, _licence_ (noun). Never American spelling.
+- **≤100 chars per line** in comment bodies.
+- **Present tense** for descriptions ("Transfers tokens…", not "Will transfer…").
+- **Precise smart-contract terminology**: reentrancy, invariant, storage slot, delegatecall,
+  selector, EIP-xxx, diamond facet.
+
+## What to cover when relevant
+
+Access control (roles, modifiers) · state mutations · invariants · gas hazards (unbounded loops,
+cold SLOADs, storage packing) · initialisation / upgrade order · events emitted · errors raised
+· side effects (external calls, transfers, mint/burn) · cross-contract interactions · pre/post
+conditions · reentrancy posture.
+
+## What to avoid
+
+Explaining Solidity itself ("this is a mapping…"). Narrating self-explanatory code ("getter that
+returns the value"). Inventing behaviour not in the code — read the implementation first.
+Referencing the current task/PR/commit — NatSpec lives with the code. Duplicating interface
+NatSpec on the implementation.
+
+## Validate before finishing
+
+For each touched file:
+
+1. **Coverage script** (heuristic, enforces the tags solhint's `use-natspec` rule flags):
+
+   ```bash
+   node .claude/skills/solidity-natspec/scripts/check_natspec.mjs <path/to/file.sol>
+   ```
+
+   Contract/interface/library need `@title` + `@author` + `@notice`. Functions/events/errors need
+   `@notice` and one `@param`/`@return` per named parameter/return with matching names. Elements
+   carrying `@inheritdoc X` pass without further tag checks. State variables are intentionally
+   not scanned — verify those by eye against "what to cover". Exit 0 means the required tags
+   exist; content quality still needs review.
+
+2. **Solhint** on the touched file(s):
+
+   ```bash
+   cd packages/ats/contracts && npx solhint <relative/path/to/file.sol>
+   ```
+
+   Any `use-natspec` warning must be resolved before reporting done. `ordering`, `gas-*`,
+   `no-unused-import` warnings are outside this skill's scope — flag them to the user instead of
+   silently editing.
+
+Insert the NatSpec directly with Edit/Write. Only emit a separate report if the user asks for a
+review-only pass.