build: tag untagged asserts for 2.90.0 release (#26607)

## Summary
- Tags all untagged assert shortcodes in preparation for the 2.90.0
client release.
- Generated by `pnpm run policy-check:asserts`.

## Merge Priority
This PR should merge **before** the version bump PR.

Author: tylerbutler

packages/dds/sequence/src/intervals/sequenceInterval.ts

@@ -307,15 +307,15 @@ export class BaseSequenceInterval implements SequenceInterval, ISerializableInte
 	 * Whether this interval overlaps the provided numerical positions.
 	 */
 	public overlapsPos(_bstart: number, _bend: number): boolean {
-		assert(false, "overlapsPos not supported on BaseSequenceInterval");
+		assert(false, 0xcd7 /* overlapsPos not supported on BaseSequenceInterval */);
 	}
 
 	public clone(): BaseSequenceInterval {
-		assert(false, "clone not supported on BaseSequenceInterval");
+		assert(false, 0xcd8 /* clone not supported on BaseSequenceInterval */);
 	}
 
 	public union(_b: BaseSequenceInterval): BaseSequenceInterval {
-		assert(false, "union not supported on BaseSequenceInterval");
+		assert(false, 0xcd9 /* union not supported on BaseSequenceInterval */);
 	}
 
 	protected verifyNotDispose(): void {
@@ -858,7 +858,7 @@ export function createTransientIntervalFromSequence(
 			endPos !== undefined &&
 			startSide !== undefined &&
 			endSide !== undefined,
-		"start and end cannot be undefined because they were not passed in as undefined",
+		0xcda /* start and end cannot be undefined because they were not passed in as undefined */,
 	);
 
 	const startSlidingPref = startReferenceSlidingPreference(

packages/dds/tree/src/shared-tree/treeCheckout.ts

@@ -505,7 +505,7 @@ export class TreeCheckout implements ITreeCheckoutFork {
 			this.labelTreeNode = node;
 		} else {
 			const current = this.currentLabelNode();
-			assert(current !== undefined, "Expected current label node to exist");
+			assert(current !== undefined, 0xcdb /* Expected current label node to exist */);
 			current.sublabels.push(node);
 		}
 	}
@@ -525,7 +525,7 @@ export class TreeCheckout implements ITreeCheckoutFork {
 		let node: LabelTree = this.labelTreeNode;
 		while (node.sublabels.length > 0) {
 			const lastChild = node.sublabels[node.sublabels.length - 1];
-			assert(lastChild !== undefined, "Expected label tree node to have children");
+			assert(lastChild !== undefined, 0xcdc /* Expected label tree node to have children */);
 			if (lastChild === this.mostRecentlyClosedLabelNode) {
 				break;
 			}
@@ -553,7 +553,7 @@ export class TreeCheckout implements ITreeCheckoutFork {
 				// Temporarily mark node as closed so currentLabelNode() returns its parent.
 				this.mostRecentlyClosedLabelNode = node;
 				const parent = this.currentLabelNode();
-				assert(parent !== undefined, "Expected parent label node to exist");
+				assert(parent !== undefined, 0xcdd /* Expected parent label node to exist */);
 				parent.sublabels.pop();
 				// Point to the parent's new last child (guaranteed closed if it exists),
 				// or undefined if the parent has no more children.

packages/dds/tree/src/text/textDomainFormatted.ts

@@ -238,7 +238,7 @@ class StringArray extends sf.array("StringArray", [() => FormattedTextAsTree.Str
 						break;
 					}
 					default: {
-						fail("Unsupported node type in text array", () => `${cursor.type}`);
+						fail(0xcde /* Unsupported node type in text array */, () => `${cursor.type}`);
 					}
 				}
 				cursor.exitNode();

packages/framework/react/src/text/formatted/quillFormattedView.tsx

@@ -360,7 +360,10 @@ const FormattedTextEditorView = React.forwardRef<
 						// The docs for retain imply this is always a number, but the type definitions allow a record here.
 						// It is unknown why the type definitions allow a record as they have no doc comments.
 						// For now this assert seems to be passing, so we just assume its always a number.
-						assert(typeof op.retain === "number", "Expected retain count to be a number");
+						assert(
+							typeof op.retain === "number",
+							0xcdf /* Expected retain count to be a number */,
+						);
 						// Convert UTF-16 retain count to codepoint count
 						const retainedStr = content.slice(utf16Pos, utf16Pos + op.retain);
 						const cpCount = codepointCount(retainedStr);

packages/runtime/container-runtime/src/opLifecycle/duplicateBatchDetector.ts

@@ -66,15 +66,15 @@ export class DuplicateBatchDetector {
 		if (otherSequenceNumber !== undefined) {
 			assert(
 				this.batchIdsBySeqNum.get(otherSequenceNumber) === batchId,
-				"batchIdToSeqNum and seqNumToBatchId should be in sync for duplicate",
+				0xce0 /* batchIdToSeqNum and seqNumToBatchId should be in sync for duplicate */,
 			);
 			return { duplicate: true, otherSequenceNumber };
 		}
 
 		// Now we know it's not a duplicate, so add it to the tracked batchIds and return.
 		assert(
 			!this.batchIdsBySeqNum.has(sequenceNumber),
-			"seqNumToBatchId and batchIdToSeqNum should be in sync",
+			0xce1 /* seqNumToBatchId and batchIdToSeqNum should be in sync */,
 		);
 
 		// Add new batch

packages/runtime/test-runtime-utils/src/assertionShortCodesMap.ts

@@ -1202,7 +1202,6 @@ export const shortCodeMap = {
 	"0x92e": "Unknown node ID",
 	"0x933": "Inverse schema changes should never be transmitted",
 	"0x934": "Expected effect to cover entire mark",
-	"0x935": "Invalid write version for FieldBatch codec",
 	"0x936": "authHeader should not be null or empty",
 	"0x93a": "Snapshot sequence number is missing",
 	"0x93c": "id compressor should be enabled",
@@ -1371,8 +1370,6 @@ export const shortCodeMap = {
 	"0xa31": "Batch must have at least one message",
 	"0xa32": "Batch must have at least one message",
 	"0xa33": "clientId (from stateHandler) could only be undefined if we've never connected, but we have a CSN so we know that's not the case",
-	"0xa34": "Should have found the batchId in batchIdBySeqNum map",
-	"0xa35": "batchIdsAll and batchIdsBySeqNum should be in sync",
 	"0xa3a": "Presence received signal without clientId",
 	"0xa3c": "Already have entry for key in map",
 	"0xa3d": "Partial batch should have exactly one message",
@@ -1765,9 +1762,7 @@ export const shortCodeMap = {
 	"0xc20": "snapshot tree not found for one of tree's summarizables",
 	"0xc21": "Forest summary content missing in snapshot",
 	"0xc22": "Not tracking a summary",
-	"0xc23": "Tracked summary properties must be available when tracking a summary",
 	"0xc24": "Already tracking a summary",
-	"0xc25": "Tracked summary properties must not be available when ready to track",
 	"0xc27": "incremental decoder not available for incremental field decoding",
 	"0xc2c": "The data store should be locally visible when generating attach summary",
 	"0xc2d": "logger would be in sessionIdOrLogger in this codepath",
@@ -1932,5 +1927,16 @@ export const shortCodeMap = {
 	"0xcd3": "equality should be symmetric",
 	"0xcd4": "identicalCompatibility should have equivalent stored schema",
 	"0xcd5": "destinationField should be unhydrated since we're in the else branch of isHydrated() check",
-	"0xcd6": "Expect coordinatorResult to be defined"
+	"0xcd6": "Expect coordinatorResult to be defined",
+	"0xcd7": "overlapsPos not supported on BaseSequenceInterval",
+	"0xcd8": "clone not supported on BaseSequenceInterval",
+	"0xcd9": "union not supported on BaseSequenceInterval",
+	"0xcda": "start and end cannot be undefined because they were not passed in as undefined",
+	"0xcdb": "Expected current label node to exist",
+	"0xcdc": "Expected label tree node to have children",
+	"0xcdd": "Expected parent label node to exist",
+	"0xcde": "Unsupported node type in text array",
+	"0xcdf": "Expected retain count to be a number",
+	"0xce0": "batchIdToSeqNum and seqNumToBatchId should be in sync for duplicate",
+	"0xce1": "seqNumToBatchId and batchIdToSeqNum should be in sync"
 };