Update policy so transient errors are not marked terminal (#26066)

This commit is contained in:
David Pierce
2026-04-28 16:01:27 +00:00
committed by GitHub
parent c17400b830
commit 54b7586106
5 changed files with 32 additions and 32 deletions
+7 -7
View File
@@ -1191,7 +1191,7 @@ their corresponding top-level category object in your `settings.json` file.
}, },
"stateTransitions": { "stateTransitions": {
"terminal": "terminal", "terminal": "terminal",
"transient": "terminal", "transient": "sticky_retry",
"not_found": "terminal", "not_found": "terminal",
"unknown": "terminal" "unknown": "terminal"
} }
@@ -1207,7 +1207,7 @@ their corresponding top-level category object in your `settings.json` file.
}, },
"stateTransitions": { "stateTransitions": {
"terminal": "terminal", "terminal": "terminal",
"transient": "terminal", "transient": "sticky_retry",
"not_found": "terminal", "not_found": "terminal",
"unknown": "terminal" "unknown": "terminal"
} }
@@ -1224,7 +1224,7 @@ their corresponding top-level category object in your `settings.json` file.
}, },
"stateTransitions": { "stateTransitions": {
"terminal": "terminal", "terminal": "terminal",
"transient": "terminal", "transient": "sticky_retry",
"not_found": "terminal", "not_found": "terminal",
"unknown": "terminal" "unknown": "terminal"
} }
@@ -1240,7 +1240,7 @@ their corresponding top-level category object in your `settings.json` file.
}, },
"stateTransitions": { "stateTransitions": {
"terminal": "terminal", "terminal": "terminal",
"transient": "terminal", "transient": "sticky_retry",
"not_found": "terminal", "not_found": "terminal",
"unknown": "terminal" "unknown": "terminal"
} }
@@ -1257,7 +1257,7 @@ their corresponding top-level category object in your `settings.json` file.
}, },
"stateTransitions": { "stateTransitions": {
"terminal": "terminal", "terminal": "terminal",
"transient": "terminal", "transient": "sticky_retry",
"not_found": "terminal", "not_found": "terminal",
"unknown": "terminal" "unknown": "terminal"
} }
@@ -1272,7 +1272,7 @@ their corresponding top-level category object in your `settings.json` file.
}, },
"stateTransitions": { "stateTransitions": {
"terminal": "terminal", "terminal": "terminal",
"transient": "terminal", "transient": "sticky_retry",
"not_found": "terminal", "not_found": "terminal",
"unknown": "terminal" "unknown": "terminal"
} }
@@ -1288,7 +1288,7 @@ their corresponding top-level category object in your `settings.json` file.
}, },
"stateTransitions": { "stateTransitions": {
"terminal": "terminal", "terminal": "terminal",
"transient": "terminal", "transient": "sticky_retry",
"not_found": "terminal", "not_found": "terminal",
"unknown": "terminal" "unknown": "terminal"
} }
@@ -56,7 +56,7 @@ describe('policyCatalog', () => {
it('marks preview transients as sticky retries', () => { it('marks preview transients as sticky retries', () => {
const [previewPolicy] = getModelPolicyChain({ previewEnabled: true }); const [previewPolicy] = getModelPolicyChain({ previewEnabled: true });
expect(previewPolicy.model).toBe(PREVIEW_GEMINI_MODEL); expect(previewPolicy.model).toBe(PREVIEW_GEMINI_MODEL);
expect(previewPolicy.stateTransitions.transient).toBe('terminal'); expect(previewPolicy.stateTransitions.transient).toBe('sticky_retry');
}); });
it('applies default actions and state transitions for unspecified kinds', () => { it('applies default actions and state transitions for unspecified kinds', () => {
@@ -50,7 +50,7 @@ export const SILENT_ACTIONS: ModelPolicyActionMap = {
const DEFAULT_STATE: ModelPolicyStateMap = { const DEFAULT_STATE: ModelPolicyStateMap = {
terminal: 'terminal', terminal: 'terminal',
transient: 'terminal', transient: 'sticky_retry',
not_found: 'terminal', not_found: 'terminal',
unknown: 'terminal', unknown: 'terminal',
}; };
@@ -557,7 +557,7 @@ export const DEFAULT_MODEL_CONFIGS: ModelConfigServiceConfig = {
}, },
stateTransitions: { stateTransitions: {
terminal: 'terminal', terminal: 'terminal',
transient: 'terminal', transient: 'sticky_retry',
not_found: 'terminal', not_found: 'terminal',
unknown: 'terminal', unknown: 'terminal',
}, },
@@ -573,7 +573,7 @@ export const DEFAULT_MODEL_CONFIGS: ModelConfigServiceConfig = {
}, },
stateTransitions: { stateTransitions: {
terminal: 'terminal', terminal: 'terminal',
transient: 'terminal', transient: 'sticky_retry',
not_found: 'terminal', not_found: 'terminal',
unknown: 'terminal', unknown: 'terminal',
}, },
@@ -590,7 +590,7 @@ export const DEFAULT_MODEL_CONFIGS: ModelConfigServiceConfig = {
}, },
stateTransitions: { stateTransitions: {
terminal: 'terminal', terminal: 'terminal',
transient: 'terminal', transient: 'sticky_retry',
not_found: 'terminal', not_found: 'terminal',
unknown: 'terminal', unknown: 'terminal',
}, },
@@ -606,7 +606,7 @@ export const DEFAULT_MODEL_CONFIGS: ModelConfigServiceConfig = {
}, },
stateTransitions: { stateTransitions: {
terminal: 'terminal', terminal: 'terminal',
transient: 'terminal', transient: 'sticky_retry',
not_found: 'terminal', not_found: 'terminal',
unknown: 'terminal', unknown: 'terminal',
}, },
@@ -623,7 +623,7 @@ export const DEFAULT_MODEL_CONFIGS: ModelConfigServiceConfig = {
}, },
stateTransitions: { stateTransitions: {
terminal: 'terminal', terminal: 'terminal',
transient: 'terminal', transient: 'sticky_retry',
not_found: 'terminal', not_found: 'terminal',
unknown: 'terminal', unknown: 'terminal',
}, },
@@ -638,7 +638,7 @@ export const DEFAULT_MODEL_CONFIGS: ModelConfigServiceConfig = {
}, },
stateTransitions: { stateTransitions: {
terminal: 'terminal', terminal: 'terminal',
transient: 'terminal', transient: 'sticky_retry',
not_found: 'terminal', not_found: 'terminal',
unknown: 'terminal', unknown: 'terminal',
}, },
@@ -654,7 +654,7 @@ export const DEFAULT_MODEL_CONFIGS: ModelConfigServiceConfig = {
}, },
stateTransitions: { stateTransitions: {
terminal: 'terminal', terminal: 'terminal',
transient: 'terminal', transient: 'sticky_retry',
not_found: 'terminal', not_found: 'terminal',
unknown: 'terminal', unknown: 'terminal',
}, },
File diff suppressed because one or more lines are too long