fix: containers without priority set should have max priority (#109)

* fix: containers without priority set should have max priority

Closes microsoft/vscode-copilot#9289

* run validation on pr

* 0.3.0-alpha.6

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 0.3.0-alpha.6
+
+- **fix:** containers without priority set should have max priority
+
 ## 0.3.0-alpha.5
 
 - **feat:** add `Expandable` elements to the renderer. See the [readme](./README.md#expandable-text) for details.

build/base.yml

@@ -4,7 +4,6 @@ trigger:
   branches:
     include:
       - main
-pr: none
 
 resources:
   repositories:

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@vscode/prompt-tsx",
-	"version": "0.3.0-alpha.5",
+	"version": "0.3.0-alpha.6",
 	"description": "Declare LLM prompts with TSX",
 	"main": "./dist/base/index.js",
 	"types": "./dist/base/index.d.ts",

src/base/promptRenderer.ts

@@ -911,7 +911,7 @@ class PromptTreeElement {
 				this._obj.props.name,
 				this._obj instanceof AssistantMessage ? this._obj.props.toolCalls : undefined,
 				this._obj instanceof ToolMessage ? this._obj.props.toolCallId : undefined,
-				this._obj.props.priority ?? 0,
+				this._obj.props.priority ?? Number.MAX_SAFE_INTEGER,
 				this._metadata,
 				this._children.map(child => child.materialize())
 			);
@@ -924,7 +924,7 @@ class PromptTreeElement {
 			return new MaterializedContainer(
 				this.id,
 				this._obj?.constructor.name,
-				this._obj?.props.priority || 0,
+				this._obj?.props.priority ?? Number.MAX_SAFE_INTEGER,
 				this._children.map(child => child.materialize()),
 				this._metadata,
 				flags

src/base/test/renderer.test.tsx

@@ -914,9 +914,8 @@ suite('PromptRenderer', () => {
 						'```',
 					].join('\n'),
 				},
-				{ role: 'user', content: 'What is your name?' },
 			]);
-			assert.equal(res2.tokenCount, 120 - BaseTokensPerCompletion);
+			assert.equal(res2.tokenCount, 108);
 		});
 	});