mirror of
https://github.com/google-gemini/gemini-cli.git
synced 2026-05-13 05:12:55 -07:00
Update policy so transient errors are not marked terminal (#26066)
This commit is contained in:
@@ -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
Reference in New Issue
Block a user