@webex/contact-center 3.12.0-llmrefactor.1 → 3.12.0-llmrefactor.11
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/.sdd/manifest.json +7 -5
- package/ai-docs/ARCHITECTURE.md +1 -1
- package/ai-docs/CONTRACTS.md +8 -3
- package/ai-docs/SECURITY.md +1 -1
- package/ai-docs/contact-center-spec.md +22 -4
- package/ai-docs/features/consult-transfer-list-policy/spec/feature-spec.md +362 -0
- package/dist/cc.js +544 -12
- package/dist/cc.js.map +1 -1
- package/dist/config.js +7 -0
- package/dist/config.js.map +1 -1
- package/dist/constants.js +10 -2
- package/dist/constants.js.map +1 -1
- package/dist/index.js.map +1 -1
- package/dist/metrics/MetricsManager.js +16 -0
- package/dist/metrics/MetricsManager.js.map +1 -1
- package/dist/metrics/behavioral-events.js +104 -0
- package/dist/metrics/behavioral-events.js.map +1 -1
- package/dist/metrics/constants.js +18 -0
- package/dist/metrics/constants.js.map +1 -1
- package/dist/services/AddressBook.js +18 -15
- package/dist/services/AddressBook.js.map +1 -1
- package/dist/services/AnswerCallOnWebexService.js +202 -0
- package/dist/services/AnswerCallOnWebexService.js.map +1 -0
- package/dist/services/EntryPoint.js +46 -68
- package/dist/services/EntryPoint.js.map +1 -1
- package/dist/services/Queue.js +27 -22
- package/dist/services/Queue.js.map +1 -1
- package/dist/services/WebexCrossClientService.js +222 -0
- package/dist/services/WebexCrossClientService.js.map +1 -0
- package/dist/services/WxAppTelephonyMercurySync.js +108 -0
- package/dist/services/WxAppTelephonyMercurySync.js.map +1 -0
- package/dist/services/config/Util.js +3 -0
- package/dist/services/config/Util.js.map +1 -1
- package/dist/services/config/constants.js +10 -6
- package/dist/services/config/constants.js.map +1 -1
- package/dist/services/config/types.js +4 -0
- package/dist/services/config/types.js.map +1 -1
- package/dist/services/core/Err.js.map +1 -1
- package/dist/services/core/WebexRequest.js +6 -2
- package/dist/services/core/WebexRequest.js.map +1 -1
- package/dist/services/core/aqm-reqs.js +28 -14
- package/dist/services/core/aqm-reqs.js.map +1 -1
- package/dist/services/core/types.js.map +1 -1
- package/dist/services/task/Task.js +73 -7
- package/dist/services/task/Task.js.map +1 -1
- package/dist/services/task/TaskFactory.js +8 -4
- package/dist/services/task/TaskFactory.js.map +1 -1
- package/dist/services/task/TaskManager.js +490 -21
- package/dist/services/task/TaskManager.js.map +1 -1
- package/dist/services/task/WebexCallingUtils.js +70 -0
- package/dist/services/task/WebexCallingUtils.js.map +1 -0
- package/dist/services/task/constants.js +4 -1
- package/dist/services/task/constants.js.map +1 -1
- package/dist/services/task/contact.js +29 -0
- package/dist/services/task/contact.js.map +1 -1
- package/dist/services/task/digital/Digital.js +3 -2
- package/dist/services/task/digital/Digital.js.map +1 -1
- package/dist/services/task/state-machine/TaskStateMachine.js +162 -53
- package/dist/services/task/state-machine/TaskStateMachine.js.map +1 -1
- package/dist/services/task/state-machine/actions.js +6 -1
- package/dist/services/task/state-machine/actions.js.map +1 -1
- package/dist/services/task/state-machine/guards.js +7 -10
- package/dist/services/task/state-machine/guards.js.map +1 -1
- package/dist/services/task/state-machine/types.js.map +1 -1
- package/dist/services/task/state-machine/uiControlsComputer.js +118 -18
- package/dist/services/task/state-machine/uiControlsComputer.js.map +1 -1
- package/dist/services/task/types.js +16 -1
- package/dist/services/task/types.js.map +1 -1
- package/dist/services/task/voice/Voice.js +456 -15
- package/dist/services/task/voice/Voice.js.map +1 -1
- package/dist/services/task/voice/WebRTC.js +4 -1
- package/dist/services/task/voice/WebRTC.js.map +1 -1
- package/dist/services/task/voice/wxAppVoiceMethods.js +354 -0
- package/dist/services/task/voice/wxAppVoiceMethods.js.map +1 -0
- package/dist/services/wxAppDiagnosticLogging.js +122 -0
- package/dist/services/wxAppDiagnosticLogging.js.map +1 -0
- package/dist/services/wxAppTelephonyUtils.js +19 -0
- package/dist/services/wxAppTelephonyUtils.js.map +1 -0
- package/dist/types/cc.d.ts +78 -4
- package/dist/types/config.d.ts +7 -0
- package/dist/types/constants.d.ts +8 -0
- package/dist/types/index.d.ts +3 -1
- package/dist/types/metrics/MetricsManager.d.ts +6 -0
- package/dist/types/metrics/constants.d.ts +17 -0
- package/dist/types/services/AddressBook.d.ts +2 -1
- package/dist/types/services/AnswerCallOnWebexService.d.ts +38 -0
- package/dist/types/services/EntryPoint.d.ts +7 -5
- package/dist/types/services/Queue.d.ts +5 -3
- package/dist/types/services/WebexCrossClientService.d.ts +32 -0
- package/dist/types/services/WxAppTelephonyMercurySync.d.ts +28 -0
- package/dist/types/services/config/constants.d.ts +8 -5
- package/dist/types/services/config/types.d.ts +17 -4
- package/dist/types/services/core/Err.d.ts +2 -0
- package/dist/types/services/core/WebexRequest.d.ts +1 -0
- package/dist/types/services/core/types.d.ts +2 -0
- package/dist/types/services/task/Task.d.ts +20 -2
- package/dist/types/services/task/TaskFactory.d.ts +2 -1
- package/dist/types/services/task/WebexCallingUtils.d.ts +11 -0
- package/dist/types/services/task/constants.d.ts +3 -0
- package/dist/types/services/task/contact.d.ts +4 -0
- package/dist/types/services/task/digital/Digital.d.ts +2 -2
- package/dist/types/services/task/state-machine/TaskStateMachine.d.ts +304 -56
- package/dist/types/services/task/state-machine/guards.d.ts +6 -5
- package/dist/types/services/task/state-machine/types.d.ts +9 -1
- package/dist/types/services/task/types.d.ts +81 -4
- package/dist/types/services/task/voice/Voice.d.ts +52 -9
- package/dist/types/services/task/voice/WebRTC.d.ts +2 -2
- package/dist/types/services/task/voice/wxAppVoiceMethods.d.ts +55 -0
- package/dist/types/services/wxAppDiagnosticLogging.d.ts +51 -0
- package/dist/types/services/wxAppTelephonyUtils.d.ts +5 -0
- package/dist/types/types.d.ts +14 -8
- package/dist/types/utils/PageCache.d.ts +20 -3
- package/dist/types.js +7 -2
- package/dist/types.js.map +1 -1
- package/dist/utils/PageCache.js +19 -5
- package/dist/utils/PageCache.js.map +1 -1
- package/dist/webex.js +1 -1
- package/package.json +8 -8
- package/src/cc.ts +657 -12
- package/src/config.ts +7 -0
- package/src/constants.ts +8 -0
- package/src/index.ts +2 -0
- package/src/metrics/MetricsManager.ts +16 -0
- package/src/metrics/ai-docs/metrics-spec.md +78 -10
- package/src/metrics/behavioral-events.ts +106 -0
- package/src/metrics/constants.ts +20 -0
- package/src/services/AddressBook.ts +17 -6
- package/src/services/AnswerCallOnWebexService.ts +239 -0
- package/src/services/EntryPoint.ts +59 -60
- package/src/services/Queue.ts +29 -12
- package/src/services/WebexCrossClientService.ts +293 -0
- package/src/services/WxAppTelephonyMercurySync.ts +142 -0
- package/src/services/agent/ai-docs/agent-spec.md +1 -0
- package/src/services/ai-docs/AGENTS.md +10 -10
- package/src/services/ai-docs/services-spec.md +6 -1
- package/src/services/config/Util.ts +3 -0
- package/src/services/config/ai-docs/AGENTS.md +1 -1
- package/src/services/config/ai-docs/ARCHITECTURE.md +2 -2
- package/src/services/config/ai-docs/config-spec.md +6 -0
- package/src/services/config/constants.ts +10 -6
- package/src/services/config/types.ts +16 -4
- package/src/services/core/Err.ts +1 -0
- package/src/services/core/WebexRequest.ts +3 -1
- package/src/services/core/ai-docs/core-spec.md +5 -1
- package/src/services/core/aqm-reqs.ts +30 -15
- package/src/services/core/types.ts +2 -0
- package/src/services/task/Task.ts +79 -8
- package/src/services/task/TaskFactory.ts +8 -3
- package/src/services/task/TaskManager.ts +716 -17
- package/src/services/task/WebexCallingUtils.ts +136 -0
- package/src/services/task/ai-docs/ARCHITECTURE.md +1 -0
- package/src/services/task/ai-docs/task-spec.md +156 -3
- package/src/services/task/constants.ts +3 -0
- package/src/services/task/contact.ts +30 -0
- package/src/services/task/digital/Digital.ts +4 -2
- package/src/services/task/state-machine/TaskStateMachine.ts +235 -81
- package/src/services/task/state-machine/actions.ts +5 -2
- package/src/services/task/state-machine/ai-docs/AGENTS.md +6 -4
- package/src/services/task/state-machine/ai-docs/ARCHITECTURE.md +35 -16
- package/src/services/task/state-machine/ai-docs/task-state-machine-spec.md +103 -33
- package/src/services/task/state-machine/guards.ts +7 -10
- package/src/services/task/state-machine/types.ts +16 -1
- package/src/services/task/state-machine/uiControlsComputer.ts +206 -32
- package/src/services/task/types.ts +114 -4
- package/src/services/task/voice/Voice.ts +595 -14
- package/src/services/task/voice/WebRTC.ts +5 -1
- package/src/services/task/voice/wxAppVoiceMethods.ts +516 -0
- package/src/services/wxAppDiagnosticLogging.ts +174 -0
- package/src/services/wxAppTelephonyUtils.ts +14 -0
- package/src/types.ts +33 -9
- package/src/utils/AGENTS.md +21 -10
- package/src/utils/PageCache.ts +38 -5
- package/src/utils/ai-docs/utils-spec.md +21 -11
- package/test/unit/spec/cc.ts +1522 -19
- package/test/unit/spec/metrics/MetricsManager.ts +25 -0
- package/test/unit/spec/metrics/behavioral-events.ts +60 -0
- package/test/unit/spec/services/AddressBook.ts +37 -6
- package/test/unit/spec/services/AnswerCallOnWebexService.ts +264 -0
- package/test/unit/spec/services/EntryPoint.ts +87 -40
- package/test/unit/spec/services/Queue.ts +123 -12
- package/test/unit/spec/services/WebexCrossClientService.ts +467 -0
- package/test/unit/spec/services/WxAppTelephonyMercurySync.ts +136 -0
- package/test/unit/spec/services/config/Util.ts +85 -0
- package/test/unit/spec/services/core/WebexRequest.ts +3 -1
- package/test/unit/spec/services/core/aqm-reqs.ts +113 -1
- package/test/unit/spec/services/task/Task.ts +274 -6
- package/test/unit/spec/services/task/TaskFactory.ts +39 -2
- package/test/unit/spec/services/task/TaskManager.ts +1717 -9
- package/test/unit/spec/services/task/WebexCallingUtils.ts +153 -0
- package/test/unit/spec/services/task/contact.ts +33 -0
- package/test/unit/spec/services/task/state-machine/TaskStateMachine.ts +1485 -109
- package/test/unit/spec/services/task/state-machine/guards.ts +210 -8
- package/test/unit/spec/services/task/state-machine/uiControlsComputer.ts +445 -7
- package/test/unit/spec/services/task/voice/Voice.ts +1412 -0
- package/test/unit/spec/services/task/voice/wxAppVoiceMethods.ts +603 -0
- package/test/unit/spec/services/wxAppDiagnosticLogging.ts +143 -0
- package/umd/contact-center.min.js +2 -2
- package/umd/contact-center.min.js.map +1 -1
package/src/types.ts
CHANGED
|
@@ -93,12 +93,19 @@ export type WebexRequestPayload = {
|
|
|
93
93
|
*/
|
|
94
94
|
type Listener = (e: string, data?: unknown) => void;
|
|
95
95
|
|
|
96
|
+
/**
|
|
97
|
+
* Mercury event handler callback.
|
|
98
|
+
* @internal
|
|
99
|
+
* @ignore
|
|
100
|
+
*/
|
|
101
|
+
type MercuryEventHandler = (data?: unknown) => void;
|
|
102
|
+
|
|
96
103
|
/**
|
|
97
104
|
* Event listener removal function type.
|
|
98
105
|
* @internal
|
|
99
106
|
* @ignore
|
|
100
107
|
*/
|
|
101
|
-
type ListenerOff = (e: string) => void;
|
|
108
|
+
type ListenerOff = (e: string, handler?: MercuryEventHandler) => void;
|
|
102
109
|
|
|
103
110
|
/**
|
|
104
111
|
* Service host configuration.
|
|
@@ -163,6 +170,8 @@ export interface CCPluginConfig {
|
|
|
163
170
|
callingClientConfig: CallingClientConfig;
|
|
164
171
|
/** Whether to skip Mobius/WebRTC registration for browser login flows */
|
|
165
172
|
disableWebRTCRegistration?: boolean;
|
|
173
|
+
/** Whether wxApp Better Together (telephony REST + usersub) is enabled at init (WXCC-6026) */
|
|
174
|
+
enableWxBetterTogether?: boolean;
|
|
166
175
|
}
|
|
167
176
|
|
|
168
177
|
/**
|
|
@@ -293,6 +302,12 @@ interface IWebexInternal {
|
|
|
293
302
|
version: string;
|
|
294
303
|
/** Calling behavior configuration */
|
|
295
304
|
callingBehavior: string;
|
|
305
|
+
/** Whether the device is registered with WDM */
|
|
306
|
+
registered?: boolean;
|
|
307
|
+
/** Register the device with WDM */
|
|
308
|
+
register?: () => Promise<void>;
|
|
309
|
+
/** Unregister the device from WDM */
|
|
310
|
+
unregister?: () => Promise<void>;
|
|
296
311
|
};
|
|
297
312
|
/** Presence service */
|
|
298
313
|
presence: unknown;
|
|
@@ -555,18 +570,16 @@ export type RequestBody =
|
|
|
555
570
|
|
|
556
571
|
/**
|
|
557
572
|
* Represents the options to fetch buddy agents for the logged in agent.
|
|
558
|
-
* Buddy agents are other agents who can be consulted or
|
|
573
|
+
* Buddy agents are other agents who can be consulted or transferred to.
|
|
559
574
|
* @public
|
|
560
575
|
* @example
|
|
561
576
|
* const opts: BuddyAgents = { mediaType: 'telephony', state: 'Available' };
|
|
562
|
-
* @ignore
|
|
563
577
|
*/
|
|
564
578
|
export type BuddyAgents = {
|
|
565
579
|
/**
|
|
566
|
-
* The media type channel to filter buddy agents.
|
|
567
|
-
* Determines which channel capability the returned agents must have.
|
|
580
|
+
* The media type channel to filter buddy agents. Defaults to telephony when omitted.
|
|
568
581
|
*/
|
|
569
|
-
mediaType
|
|
582
|
+
mediaType?: 'telephony' | 'chat' | 'social' | 'email';
|
|
570
583
|
|
|
571
584
|
/**
|
|
572
585
|
* Optional filter for agent state.
|
|
@@ -574,6 +587,11 @@ export type BuddyAgents = {
|
|
|
574
587
|
* If omitted, returns both available and idle agents.
|
|
575
588
|
*/
|
|
576
589
|
state?: 'Available' | 'Idle';
|
|
590
|
+
|
|
591
|
+
/**
|
|
592
|
+
* Applies the default state policy for the operation. An explicit state takes precedence.
|
|
593
|
+
*/
|
|
594
|
+
action?: 'Consult' | 'Transfer';
|
|
577
595
|
};
|
|
578
596
|
|
|
579
597
|
/**
|
|
@@ -587,11 +605,15 @@ export type ConfigFlags = {
|
|
|
587
605
|
webRtcEnabled: boolean;
|
|
588
606
|
autoWrapup: boolean;
|
|
589
607
|
aiFeature?: AIFeatureFlags;
|
|
608
|
+
/** Agent-profile policy used to derive task consult/transfer destination controls. */
|
|
609
|
+
consultTransfer?: Contact.ConsultTransferDestinationConfig;
|
|
590
610
|
/**
|
|
591
611
|
* Optional toggle to globally enable/disable recording controls.
|
|
592
612
|
* Falls back to backend hints when omitted.
|
|
593
613
|
*/
|
|
594
614
|
isRecordingEnabled?: boolean;
|
|
615
|
+
/** Whether wxApp thick-client answer controls/APIs are enabled (WXCC-6026) */
|
|
616
|
+
enableWxBetterTogether?: boolean;
|
|
595
617
|
};
|
|
596
618
|
|
|
597
619
|
/**
|
|
@@ -681,10 +703,12 @@ export interface AddressBookEntrySearchParams extends BaseSearchParams {
|
|
|
681
703
|
export interface EntryPointRecord {
|
|
682
704
|
id: string;
|
|
683
705
|
name: string;
|
|
706
|
+
/** Dialled number mapped to this entry point when included in the list response. */
|
|
707
|
+
number?: string;
|
|
684
708
|
description?: string;
|
|
685
|
-
type
|
|
686
|
-
isActive
|
|
687
|
-
orgId
|
|
709
|
+
type?: string;
|
|
710
|
+
isActive?: boolean;
|
|
711
|
+
orgId?: string;
|
|
688
712
|
createdAt?: string;
|
|
689
713
|
updatedAt?: string;
|
|
690
714
|
settings?: Record<string, any>;
|
package/src/utils/AGENTS.md
CHANGED
|
@@ -12,7 +12,7 @@ The utils scope currently provides shared pagination and cache behavior for cont
|
|
|
12
12
|
|
|
13
13
|
- **Typed Pagination Contracts**: Reusable interfaces for response metadata and query params
|
|
14
14
|
- **Generic In-Memory Page Caching**: `PageCache<T>` utility for simple pagination reuse
|
|
15
|
-
- **Cache Safety Rules**: Explicit bypass behavior for
|
|
15
|
+
- **Cache Safety Rules**: Explicit bypass behavior for query variants and enabled result/response-shape flags
|
|
16
16
|
- **Spec-Driven Utility Workflow**: Utility-specific implementation and validation flow documented inline in this file
|
|
17
17
|
|
|
18
18
|
| Component | File | Description |
|
|
@@ -36,7 +36,7 @@ src/utils/
|
|
|
36
36
|
|
|
37
37
|
## PageCache Utility
|
|
38
38
|
|
|
39
|
-
`PageCache<T>` provides a consistent caching model for paginated list APIs. It is optimized for simple page browsing and intentionally bypasses cache for parameterized query cases (`search`, `filter`, `attributes`, `sortBy`).
|
|
39
|
+
`PageCache<T>` provides a consistent caching model for paginated list APIs. It is optimized for simple page browsing and intentionally bypasses cache for parameterized query cases (`search`, `filter`, `attributes`, `sortBy`) and enabled result/response-shape flags (`desktopProfileFilter`, `provisioningView`, `singleObjectResponse`).
|
|
40
40
|
|
|
41
41
|
### Reference Usage
|
|
42
42
|
|
|
@@ -49,8 +49,16 @@ const page = PAGINATION_DEFAULTS.PAGE;
|
|
|
49
49
|
const pageSize = PAGINATION_DEFAULTS.PAGE_SIZE;
|
|
50
50
|
const cacheKey = cache.buildCacheKey(scopeId, page, pageSize);
|
|
51
51
|
|
|
52
|
-
// Include
|
|
53
|
-
const canUseCache = cache.canUseCache({
|
|
52
|
+
// Include every supported result- or response-shape-changing input.
|
|
53
|
+
const canUseCache = cache.canUseCache({
|
|
54
|
+
search,
|
|
55
|
+
filter,
|
|
56
|
+
attributes,
|
|
57
|
+
sortBy,
|
|
58
|
+
desktopProfileFilter,
|
|
59
|
+
provisioningView,
|
|
60
|
+
singleObjectResponse,
|
|
61
|
+
});
|
|
54
62
|
|
|
55
63
|
if (canUseCache) {
|
|
56
64
|
const cachedEntry = cache.getCachedPage(cacheKey);
|
|
@@ -80,7 +88,7 @@ return response;
|
|
|
80
88
|
```mermaid
|
|
81
89
|
graph TD
|
|
82
90
|
A[Request arrives with scopeId/page/pageSize] --> B{canUseCache?}
|
|
83
|
-
B -->|No:
|
|
91
|
+
B -->|No: query variant or enabled result/shape flag| C[Bypass cache and call API]
|
|
84
92
|
B -->|Yes| D[buildCacheKey scopeId:page:pageSize]
|
|
85
93
|
D --> E["getCachedPage(cacheKey)"]
|
|
86
94
|
E -->|Miss| C
|
|
@@ -138,11 +146,14 @@ Contract passed to `canUseCache()`:
|
|
|
138
146
|
- `filter?: string`
|
|
139
147
|
- `attributes?: string`
|
|
140
148
|
- `sortBy?: string`
|
|
149
|
+
- `desktopProfileFilter?: boolean`
|
|
150
|
+
- `provisioningView?: boolean`
|
|
151
|
+
- `singleObjectResponse?: boolean`
|
|
141
152
|
|
|
142
153
|
Behavior note:
|
|
143
154
|
|
|
144
|
-
- Cache bypass is triggered by
|
|
145
|
-
-
|
|
155
|
+
- Cache bypass is triggered by a truthy query variant or enabled result/response-shape flag; an explicit `false` boolean remains cache-compatible because it is semantically equivalent to omission.
|
|
156
|
+
- `sortOrder` is not a `CacheValidationParams` field. A service where it changes the wire order without `sortBy` must pass an effective `sortBy`; Queue normalizes a direction-only request to `sortBy: 'name'` before cache validation.
|
|
146
157
|
|
|
147
158
|
### `BaseSearchParams`
|
|
148
159
|
|
|
@@ -175,8 +186,9 @@ Returns `true` only when all of these are absent:
|
|
|
175
186
|
- `filter`
|
|
176
187
|
- `attributes`
|
|
177
188
|
- `sortBy`
|
|
189
|
+
- `desktopProfileFilter`, `provisioningView`, or `singleObjectResponse` when `true`
|
|
178
190
|
|
|
179
|
-
`
|
|
191
|
+
An explicit `false` boolean remains eligible. `sortOrder` is normalized by the consuming service when it changes the effective wire order; Queue passes `sortBy: 'name'` for a direction-only request.
|
|
180
192
|
|
|
181
193
|
### `buildCacheKey(scopeId: string, page: number, pageSize: number): string`
|
|
182
194
|
|
|
@@ -223,7 +235,6 @@ Current consumers of `PageCache` and defaults:
|
|
|
223
235
|
| Consumer | File | Usage |
|
|
224
236
|
| ----------------------- | ---------------------------------------------------------- | ------------------------------------------------------------ |
|
|
225
237
|
| `AddressBook` | [`../services/AddressBook.ts`](../services/AddressBook.ts) | Caches paged address-book responses |
|
|
226
|
-
| `EntryPoint` | [`../services/EntryPoint.ts`](../services/EntryPoint.ts) | Caches paged entry-point responses |
|
|
227
238
|
| `Queue` | [`../services/Queue.ts`](../services/Queue.ts) | Caches paged queue responses |
|
|
228
239
|
| `Public type contracts` | [`../types.ts`](../types.ts) | Re-exports pagination/search contracts into SDK-facing types |
|
|
229
240
|
|
|
@@ -268,7 +279,7 @@ flowchart TD
|
|
|
268
279
|
- [ ] `canUseCache()` bypass conditions are unchanged or intentionally updated
|
|
269
280
|
- [ ] `totalRecords`/`totalItems` mapping behavior is preserved
|
|
270
281
|
- [ ] Logging still uses `LoggerProxy` with `module` and `method`
|
|
271
|
-
- [ ] AddressBook/
|
|
282
|
+
- [ ] AddressBook/Queue cache integration remains correct; EntryPoint remains explicitly uncached
|
|
272
283
|
|
|
273
284
|
---
|
|
274
285
|
|
package/src/utils/PageCache.ts
CHANGED
|
@@ -97,6 +97,12 @@ export interface CacheValidationParams {
|
|
|
97
97
|
attributes?: string;
|
|
98
98
|
/** Sort by parameter */
|
|
99
99
|
sortBy?: string;
|
|
100
|
+
/** Desktop-profile filtering changes the returned result set */
|
|
101
|
+
desktopProfileFilter?: boolean;
|
|
102
|
+
/** Provisioning-view filtering changes the returned result set */
|
|
103
|
+
provisioningView?: boolean;
|
|
104
|
+
/** Single-object response mode changes the response shape */
|
|
105
|
+
singleObjectResponse?: boolean;
|
|
100
106
|
}
|
|
101
107
|
|
|
102
108
|
/**
|
|
@@ -116,8 +122,18 @@ const DEFAULT_CACHE_TTL_MINUTES = 5;
|
|
|
116
122
|
* // Create a cache instance for a specific data type
|
|
117
123
|
* const cache = new PageCache<AddressBookEntry>('AddressBook');
|
|
118
124
|
*
|
|
119
|
-
* // Check if we can use cache
|
|
120
|
-
* if (
|
|
125
|
+
* // Check if we can use cache for the complete result/response-shape input set
|
|
126
|
+
* if (
|
|
127
|
+
* cache.canUseCache({
|
|
128
|
+
* search,
|
|
129
|
+
* filter,
|
|
130
|
+
* attributes,
|
|
131
|
+
* sortBy,
|
|
132
|
+
* desktopProfileFilter,
|
|
133
|
+
* provisioningView,
|
|
134
|
+
* singleObjectResponse,
|
|
135
|
+
* })
|
|
136
|
+
* ) {
|
|
121
137
|
* const cacheKey = cache.buildCacheKey(orgId, page, pageSize);
|
|
122
138
|
* const cachedPage = cache.getCachedPage(cacheKey);
|
|
123
139
|
*
|
|
@@ -145,15 +161,32 @@ export class PageCache<T> {
|
|
|
145
161
|
|
|
146
162
|
/**
|
|
147
163
|
* Checks if cache can be used for the given parameters.
|
|
148
|
-
* Cache is only used for simple pagination without
|
|
164
|
+
* Cache is only used for simple pagination without query variants or enabled
|
|
165
|
+
* result/response-shape flags.
|
|
149
166
|
* @param {CacheValidationParams} params - Parameters to validate
|
|
150
167
|
* @returns {boolean} True if cache can be used
|
|
151
168
|
* @public
|
|
152
169
|
*/
|
|
153
170
|
public canUseCache(params: CacheValidationParams): boolean {
|
|
154
|
-
const {
|
|
171
|
+
const {
|
|
172
|
+
search,
|
|
173
|
+
filter,
|
|
174
|
+
attributes,
|
|
175
|
+
sortBy,
|
|
176
|
+
desktopProfileFilter,
|
|
177
|
+
provisioningView,
|
|
178
|
+
singleObjectResponse,
|
|
179
|
+
} = params;
|
|
155
180
|
|
|
156
|
-
return
|
|
181
|
+
return (
|
|
182
|
+
!search &&
|
|
183
|
+
!filter &&
|
|
184
|
+
!attributes &&
|
|
185
|
+
!sortBy &&
|
|
186
|
+
!desktopProfileFilter &&
|
|
187
|
+
!provisioningView &&
|
|
188
|
+
!singleObjectResponse
|
|
189
|
+
);
|
|
157
190
|
}
|
|
158
191
|
|
|
159
192
|
/**
|
|
@@ -31,7 +31,7 @@ The utils scope currently provides shared pagination and cache behavior for cont
|
|
|
31
31
|
|
|
32
32
|
- **Generic In-Memory Page Caching**: `PageCache<T>` utility for simple pagination reuse
|
|
33
33
|
|
|
34
|
-
- **Cache Safety Rules**: Explicit bypass behavior for search/filter/sort
|
|
34
|
+
- **Cache Safety Rules**: Explicit bypass behavior for search/filter/sort and enabled result/shape flags
|
|
35
35
|
|
|
36
36
|
- **Spec-Driven Utility Workflow**: Utility-specific implementation and validation flow documented inline in this file
|
|
37
37
|
|
|
@@ -88,7 +88,7 @@ No claim is made that `src/types.ts`, AddressBook, EntryPoint, or Queue is imple
|
|
|
88
88
|
## Requirements
|
|
89
89
|
| ID | WHAT | WHY | Source Evidence | Test / Example Evidence | Assumptions / Gaps | Confidence |
|
|
90
90
|
|---|---|---|---|---|---|---|
|
|
91
|
-
| UTILS-R-001 | Use cache only for simple pagination requests without search, filter, attributes, or
|
|
91
|
+
| UTILS-R-001 | Use cache only for simple pagination requests without search, filter, attributes, sortBy, or enabled desktop-profile/provisioning/single-object flags. Explicit `false` flags remain cache-compatible because they do not change the base result or response shape. | Parameterized or enabled variant queries cannot safely reuse a page keyed only by scope/page/pageSize, while a disabled boolean is semantically the same as omission. | `src/utils/PageCache.ts` | `test/unit/spec/services/AddressBook.ts`, `test/unit/spec/services/Queue.ts` | None. | PRESENT |
|
|
92
92
|
| UTILS-R-002 | Build cache keys from a caller-defined `scopeId` plus `page:pageSize`: `orgId` for EntryPoint/Queue and `bookId` for AddressBook. | Consumer scope and page boundaries prevent cross-scope or cross-page reuse. | `src/utils/PageCache.ts`, `src/services/AddressBook.ts`, `src/services/EntryPoint.ts`, `src/services/Queue.ts` | `test/unit/spec/services/AddressBook.ts`, `test/unit/spec/services/EntryPoint.ts`, `test/unit/spec/services/Queue.ts` | `PageCache.buildCacheKey` names its first implementation parameter `orgId`, but runtime callers establish the broader `scopeId` semantics. | PRESENT |
|
|
93
93
|
| UTILS-R-003 | Expire entries after the configured five-minute TTL and delete them on stale lookup. | Bounded staleness prevents indefinite reuse of remote service data. | `src/utils/PageCache.ts` | None | Direct fake-clock expiration coverage is absent from current consumer tests. | PRESENT |
|
|
94
94
|
| UTILS-R-004 | Cache data with total-page/record metadata and allow owning consumers to clear the cache. | Paginated services need consistent metadata without transferring ownership of remote records to the SDK. | `src/utils/PageCache.ts` | `test/unit/spec/services/AddressBook.ts` | Cache hit/miss and metadata are exercised through consumers; `clearCache()` has no direct assertion. | PRESENT |
|
|
@@ -96,7 +96,7 @@ No claim is made that `src/types.ts`, AddressBook, EntryPoint, or Queue is imple
|
|
|
96
96
|
| UTILS-R-006 | Keep PageCache free of rollout flags; consuming services decide whether to invoke it and query parameters determine cache eligibility. | Cache correctness must depend on request shape, not hidden deployment state. | `src/utils/PageCache.ts` | `test/unit/spec/services/AddressBook.ts`, `test/unit/spec/services/Queue.ts` | None; rollout applicability is explicitly N/A. | PRESENT |
|
|
97
97
|
|
|
98
98
|
## Design Overview
|
|
99
|
-
Utils separates its stable consumption boundary from collaborators so ownership and failure behavior stay explicit. Caching is intentionally limited to simple page browsing; search/filter/sort requests bypass cache to prevent incorrect reuse.
|
|
99
|
+
Utils separates its stable consumption boundary from collaborators so ownership and failure behavior stay explicit. Caching is intentionally limited to simple page browsing; search/filter/sort requests and enabled result/shape flags bypass cache to prevent incorrect reuse.
|
|
100
100
|
|
|
101
101
|
> **This is the authoritative documentation for the `src/utils` scope.** It covers shared pagination/cache contracts used by data services. For task routing and cross-service conventions, see the [root orchestrator AGENTS.md](../../../AGENTS.md).
|
|
102
102
|
|
|
@@ -117,7 +117,7 @@ Cross-scope mention:
|
|
|
117
117
|
```mermaid
|
|
118
118
|
flowchart LR
|
|
119
119
|
Service[AddressBook / EntryPoint / Queue] --> Eligible{canUseCache params?}
|
|
120
|
-
Eligible -->|
|
|
120
|
+
Eligible -->|query variant or enabled result/shape flag| Backend[Fetch without cache]
|
|
121
121
|
Eligible -->|simple pagination| Key[buildCacheKey scopeId:page:pageSize]
|
|
122
122
|
Key --> Lookup[getCachedPage]
|
|
123
123
|
Lookup -->|fresh| Hit[Return cached page]
|
|
@@ -201,7 +201,7 @@ classDiagram
|
|
|
201
201
|
```
|
|
202
202
|
|
|
203
203
|
## Use Cases
|
|
204
|
-
- **UC-1 Cache eligibility:** AddressBook, EntryPoint, or Queue bypasses cache whenever search, filter, attributes, or
|
|
204
|
+
- **UC-1 Cache eligibility:** AddressBook, EntryPoint, or Queue bypasses cache whenever search, filter, attributes, sort, or an enabled desktop-profile/provisioning/single-object flag is supplied; explicit `false` flags stay eligible. Evidence: `src/utils/PageCache.ts`, `test/unit/spec/services/AddressBook.ts`, `test/unit/spec/services/Queue.ts`.
|
|
205
205
|
- **UC-2 Cache-key construction:** a simple page lookup uses `<scopeId>:page:pageSize`; AddressBook supplies `bookId`, while EntryPoint and Queue supply `orgId`. Evidence: `src/utils/PageCache.ts`, `src/services/AddressBook.ts`, `src/services/EntryPoint.ts`, `src/services/Queue.ts`.
|
|
206
206
|
- **UC-3 TTL lookup/expiry:** a fresh cached entry is returned; an expired entry is deleted and treated as a miss. Evidence: `src/utils/PageCache.ts`; no direct expiration test currently exists.
|
|
207
207
|
- **UC-4 Page insertion and clearing:** successful page data and normalized totals are cached, while `clearCache()` removes all entries for that service instance. Evidence: `src/utils/PageCache.ts`; consumer tests cover insertion/hit behavior, but not direct clearing.
|
|
@@ -209,7 +209,7 @@ classDiagram
|
|
|
209
209
|
## State Model
|
|
210
210
|
Utils retains only in-memory runtime state. Durable domain records remain owned by remote Webex services. State changes are driven by explicit calls, events, timers, or actor transitions documented below.
|
|
211
211
|
|
|
212
|
-
`PageCache<T>` provides a consistent caching model for paginated list APIs. It is optimized for simple page browsing and intentionally bypasses cache for parameterized query cases (`search`, `filter`, `attributes`, `sortBy`).
|
|
212
|
+
`PageCache<T>` provides a consistent caching model for paginated list APIs. It is optimized for simple page browsing and intentionally bypasses cache for parameterized query cases (`search`, `filter`, `attributes`, `sortBy`) and enabled result/shape flags (`desktopProfileFilter`, `provisioningView`, `singleObjectResponse`).
|
|
213
213
|
|
|
214
214
|
```typescript
|
|
215
215
|
import PageCache, {PAGINATION_DEFAULTS} from '../utils/PageCache';
|
|
@@ -222,8 +222,16 @@ const pageSize = PAGINATION_DEFAULTS.PAGE_SIZE;
|
|
|
222
222
|
const scopeId = bookIdOrOrgId;
|
|
223
223
|
const cacheKey = cache.buildCacheKey(scopeId, page, pageSize);
|
|
224
224
|
|
|
225
|
-
// Include
|
|
226
|
-
const canUseCache = cache.canUseCache({
|
|
225
|
+
// Include every supported result- or shape-changing input.
|
|
226
|
+
const canUseCache = cache.canUseCache({
|
|
227
|
+
search,
|
|
228
|
+
filter,
|
|
229
|
+
attributes,
|
|
230
|
+
sortBy,
|
|
231
|
+
desktopProfileFilter,
|
|
232
|
+
provisioningView,
|
|
233
|
+
singleObjectResponse,
|
|
234
|
+
});
|
|
227
235
|
|
|
228
236
|
if (canUseCache) {
|
|
229
237
|
const cachedEntry = cache.getCachedPage(cacheKey);
|
|
@@ -251,7 +259,7 @@ return response;
|
|
|
251
259
|
```mermaid
|
|
252
260
|
graph TD
|
|
253
261
|
A[Request arrives with scopeId/page/pageSize] --> B{canUseCache?}
|
|
254
|
-
B -->|No:
|
|
262
|
+
B -->|No: query variant or enabled result/shape flag| C[Bypass cache and call API]
|
|
255
263
|
B -->|Yes| D[buildCacheKey scopeId:page:pageSize]
|
|
256
264
|
D --> E["getCachedPage(cacheKey)"]
|
|
257
265
|
E -->|Miss| C
|
|
@@ -277,7 +285,9 @@ Returns `true` only when all of these are absent:
|
|
|
277
285
|
|
|
278
286
|
- `sortBy`
|
|
279
287
|
|
|
280
|
-
`
|
|
288
|
+
- `desktopProfileFilter`, `provisioningView`, or `singleObjectResponse` when `true`
|
|
289
|
+
|
|
290
|
+
An explicit `false` boolean remains eligible because it is semantically identical to omitting that flag. Services must pass an effective `sortBy` whenever `sortOrder` changes the wire order; Queue normalizes a direction-only request to `sortBy=name` before cache validation.
|
|
281
291
|
|
|
282
292
|
Builds deterministic cache key format:
|
|
283
293
|
|
|
@@ -320,7 +330,7 @@ Note: `clearCache()` and `getCacheSize()` are available for future use and are n
|
|
|
320
330
|
|
|
321
331
|
## Pitfalls
|
|
322
332
|
- Do not describe the first cache-key field as universally `orgId`: AddressBook passes `bookId`, while EntryPoint and Queue pass `orgId`.
|
|
323
|
-
- Do not cache queries containing `search`, `filter`, `attributes`,
|
|
333
|
+
- Do not cache queries containing `search`, `filter`, `attributes`, `sortBy`, or enabled desktop-profile/provisioning/single-object flags; their result set or shape is not represented in the key. Explicit `false` flags do not create a variant.
|
|
324
334
|
- Do not treat an expired entry as usable data. `getCachedPage` deletes it and returns `null`.
|
|
325
335
|
- Do not cache a rejected request or a response without data; only the consuming service owns the backend request and decides when to call `cachePage`.
|
|
326
336
|
|