@openstax/ts-utils 1.20.1 → 1.20.3

package/dist/cjs/misc/helpers.d.ts

@@ -80,6 +80,8 @@ export declare const mapFind: <I, R>(array: I[], mapper: (item: I) => R, predica
 /**
  * returns a function that will only ever call the given function once, returning the first result for every subsequent call
  *
+ * does not cache rejected promises, to avoid cache poisoning on failed async requests
+ *
  * eg:
  *   const heavyFunction = () => 'hello';
  *   const butOnlyOnce = once(() => 'hello');
@@ -90,6 +92,8 @@ export declare const mapFind: <I, R>(array: I[], mapper: (item: I) => R, predica
 export declare const once: <F extends (...args: any[]) => any>(fn: F) => F;
 /**
  * memoizes a function with any number of arguments
+ *
+ * does not cache rejected promises, to avoid cache poisoning on failed async requests
  */
 export declare const memoize: <F extends (...args: any[]) => any>(fn: F) => F;
 /**

package/dist/cjs/misc/helpers.js

@@ -107,6 +107,8 @@ exports.mapFind = mapFind;
 /**
  * returns a function that will only ever call the given function once, returning the first result for every subsequent call
  *
+ * does not cache rejected promises, to avoid cache poisoning on failed async requests
+ *
  * eg:
  *   const heavyFunction = () => 'hello';
  *   const butOnlyOnce = once(() => 'hello');
@@ -133,6 +135,8 @@ const once = (fn) => {
 exports.once = once;
 /**
  * memoizes a function with any number of arguments
+ *
+ * does not cache rejected promises, to avoid cache poisoning on failed async requests
  */
 const memoize = (fn) => {
     const cache = {};
@@ -155,7 +159,15 @@ const memoize = (fn) => {
     };
     return ((...args) => {
         const thisCache = resolveCache(cache, args);
-        return thisCache.result = (thisCache.result || fn(...args));
+        if ('result' in thisCache) {
+            return thisCache.result;
+        }
+        thisCache.result = fn(...args);
+        if (typeof thisCache.result === 'object' && thisCache.result instanceof Promise) {
+            // Clear the result when possible but do not return a Promise that resolves to the initialValue
+            thisCache.result.catch(() => delete thisCache.result);
+        }
+        return thisCache.result;
     });
 };
 exports.memoize = memoize;

package/dist/cjs/services/lrsGateway/addStatementDefaultFields.d.ts

@@ -1,5 +1,5 @@
-import { XapiStatement, EagerXapiStatement } from ".";
-import { User } from "../authProvider";
-export declare const addStatementDefaultFields: (statement: Pick<XapiStatement, 'object' | 'verb' | 'context' | 'result'> & {
+import { User } from '../authProvider';
+import { EagerXapiStatement, UXapiStatement, XapiStatement } from '.';
+export declare const addStatementDefaultFields: (statement: (Pick<XapiStatement, 'object' | 'verb' | 'context' | 'result'> & {
     id?: string;
-}, user: User) => EagerXapiStatement;
+}) | UXapiStatement, user: User) => EagerXapiStatement;

package/dist/cjs/services/lrsGateway/addStatementDefaultFields.js

@@ -8,7 +8,6 @@ const formatISO_1 = __importDefault(require("date-fns/formatISO"));
 const uuid_1 = require("uuid");
 const addStatementDefaultFields = (statement, user) => ({
     id: (0, uuid_1.v4)(),
-    ...statement,
     actor: {
         account: {
             homePage: 'https://openstax.org',
@@ -16,6 +15,7 @@ const addStatementDefaultFields = (statement, user) => ({
         },
         objectType: 'Agent',
     },
-    timestamp: (0, formatISO_1.default)(new Date())
+    timestamp: (0, formatISO_1.default)(new Date()),
+    ...statement,
 });
 exports.addStatementDefaultFields = addStatementDefaultFields;

package/dist/cjs/services/lrsGateway/attempt-utils.d.ts

@@ -1,23 +1,23 @@
-import { LrsGateway, XapiStatement } from '.';
+import { LrsGateway, UXapiStatement } from '.';
 export declare type ActivityState = {
     attempts: number;
     completedAttempts: number;
-    currentAttempt?: XapiStatement;
-    currentAttemptCompleted?: XapiStatement;
-    currentAttemptStatements: XapiStatement[];
-    mostRecentAttemptWithCompleted?: XapiStatement;
-    mostRecentAttemptWithCompletedCompleted?: XapiStatement;
+    currentAttempt?: UXapiStatement;
+    currentAttemptCompleted?: UXapiStatement;
+    currentAttemptStatements: UXapiStatement[];
+    mostRecentAttemptWithCompleted?: UXapiStatement;
+    mostRecentAttemptWithCompletedCompleted?: UXapiStatement;
 };
-export declare const matchAttempt: (statement: XapiStatement) => boolean;
-export declare const matchAttemptCompleted: (attempt: XapiStatement) => (statement: XapiStatement) => boolean;
-export declare const resolveAttempts: (statements: XapiStatement[], options?: {
+export declare const matchAttempt: (statement: UXapiStatement) => boolean;
+export declare const matchAttemptCompleted: (attempt: UXapiStatement) => (statement: UXapiStatement) => boolean;
+export declare const resolveAttempts: (statements: UXapiStatement[], options?: {
     activityIRI?: string | undefined;
     parentActivityAttempt?: string | undefined;
-} | undefined) => XapiStatement[];
-export declare const resolveCompletedForAttempt: (statements: XapiStatement[], attempt: XapiStatement, activityIRI?: string | undefined) => XapiStatement | undefined;
-export declare const oldestStatement: (statements: XapiStatement[]) => XapiStatement | undefined;
-export declare const mostRecentStatement: (statements: XapiStatement[]) => XapiStatement | undefined;
-export declare const resolveAttemptInfo: (statements: XapiStatement[], options?: {
+} | undefined) => UXapiStatement[];
+export declare const resolveCompletedForAttempt: (statements: UXapiStatement[], attempt: UXapiStatement, activityIRI?: string | undefined) => UXapiStatement | undefined;
+export declare const oldestStatement: (statements: UXapiStatement[]) => UXapiStatement | undefined;
+export declare const mostRecentStatement: (statements: UXapiStatement[]) => UXapiStatement | undefined;
+export declare const resolveAttemptInfo: (statements: UXapiStatement[], options?: {
     activityIRI?: string | undefined;
     currentAttempt?: string | undefined;
     parentActivityAttempt?: string | undefined;
@@ -28,20 +28,20 @@ export declare const loadStatementsForActivityAndFirstChildren: (gateway: LrsGat
     attempt?: string | undefined;
     ensureSync?: boolean | undefined;
     user?: string | undefined;
-} | undefined) => Promise<XapiStatement[]>;
+} | undefined) => Promise<import(".").XapiStatement[]>;
 export declare const loadActivityAttemptInfo: (gateway: LrsGateway, activityIRI: string, options?: {
     currentAttempt?: string | undefined;
     parentActivityAttempt?: string | undefined;
     ensureSync?: boolean | undefined;
 } | undefined) => Promise<ActivityState>;
-export declare const createStatement: (verb: XapiStatement['verb'], activity: {
+export declare const createStatement: (verb: UXapiStatement['verb'], activity: {
     iri: string;
     type: string;
     name: string;
     extensions?: {
         [key: string]: string;
     } | undefined;
-}, attempt: string, parentActivityIRI?: string | undefined) => Pick<XapiStatement, 'object' | 'verb' | 'context'>;
+}, attempt: string, parentActivityIRI?: string | undefined) => Pick<UXapiStatement, 'object' | 'verb' | 'context'>;
 export declare const createAttemptStatement: (activity: {
     iri: string;
     type: string;
@@ -52,7 +52,7 @@ export declare const createAttemptStatement: (activity: {
 }, parentActivity?: {
     iri?: string | undefined;
     attempt?: string | undefined;
-} | undefined) => Pick<XapiStatement, 'object' | 'verb' | 'context'>;
+} | undefined) => Pick<UXapiStatement, 'object' | 'verb' | 'context'>;
 export declare const putAttemptStatement: (gateway: LrsGateway, activity: {
     iri: string;
     type: string;
@@ -64,7 +64,7 @@ export declare const putAttemptStatement: (gateway: LrsGateway, activity: {
     iri?: string | undefined;
     attempt?: string | undefined;
 } | undefined) => Promise<import(".").EagerXapiStatement>;
-export declare const createAttemptActivityStatement: (attemptStatement: XapiStatement, verb: XapiStatement['verb'], result?: XapiStatement['result']) => Pick<XapiStatement, 'object' | 'verb' | 'context' | 'result'>;
-export declare const putAttemptActivityStatement: (gateway: LrsGateway, attemptStatement: XapiStatement, verb: XapiStatement['verb'], result?: XapiStatement['result']) => Promise<import(".").EagerXapiStatement>;
-export declare const createCompletedStatement: (attemptStatement: XapiStatement, result?: XapiStatement['result']) => Pick<XapiStatement, 'object' | 'verb' | 'context' | 'result'>;
-export declare const putCompletedStatement: (gateway: LrsGateway, attemptStatement: XapiStatement, result: XapiStatement['result']) => Promise<import(".").EagerXapiStatement>;
+export declare const createAttemptActivityStatement: (attemptStatement: UXapiStatement, verb: UXapiStatement['verb'], result?: UXapiStatement['result']) => Pick<UXapiStatement, 'object' | 'verb' | 'context' | 'result'>;
+export declare const putAttemptActivityStatement: (gateway: LrsGateway, attemptStatement: UXapiStatement, verb: UXapiStatement['verb'], result?: UXapiStatement['result']) => Promise<import(".").EagerXapiStatement>;
+export declare const createCompletedStatement: (attemptStatement: UXapiStatement, result?: UXapiStatement['result']) => Pick<UXapiStatement, 'object' | 'verb' | 'context' | 'result'>;
+export declare const putCompletedStatement: (gateway: LrsGateway, attemptStatement: UXapiStatement, result: UXapiStatement['result']) => Promise<import(".").EagerXapiStatement>;

package/dist/cjs/services/lrsGateway/attempt-utils.js

@@ -42,9 +42,9 @@ exports.resolveAttempts = resolveAttempts;
 const resolveCompletedForAttempt = (statements, attempt, activityIRI) => statements.find(statement => (0, exports.matchAttemptCompleted)(attempt)(statement)
     && (!activityIRI || statement.object.id === activityIRI));
 exports.resolveCompletedForAttempt = resolveCompletedForAttempt;
-const oldestStatement = (statements) => statements.reduce((result, statement) => result && (0, isBefore_1.default)((0, parseISO_1.default)(result.stored || result.timestamp), (0, parseISO_1.default)(statement.timestamp)) ? result : statement, statements[0]);
+const oldestStatement = (statements) => statements.reduce((result, statement) => result && (0, isBefore_1.default)((0, parseISO_1.default)('stored' in result && result.stored ? result.stored : result.timestamp), (0, parseISO_1.default)(statement.timestamp)) ? result : statement, statements[0]);
 exports.oldestStatement = oldestStatement;
-const mostRecentStatement = (statements) => statements.reduce((result, statement) => result && (0, isAfter_1.default)((0, parseISO_1.default)(result.stored || result.timestamp), (0, parseISO_1.default)(statement.timestamp)) ? result : statement, statements[0]);
+const mostRecentStatement = (statements) => statements.reduce((result, statement) => result && (0, isAfter_1.default)((0, parseISO_1.default)('stored' in result && result.stored ? result.stored : result.timestamp), (0, parseISO_1.default)(statement.timestamp)) ? result : statement, statements[0]);
 exports.mostRecentStatement = mostRecentStatement;
 const resolveAttemptInfo = (statements, options) => {
     // TODO optimize. i'm 100% that this could all be done in one iteration but i'm not messing around with that for now.

package/dist/cjs/services/lrsGateway/index.d.ts

@@ -74,6 +74,7 @@ export interface XapiStatement {
 }
 export declare type SavedXapiStatement = WithRequired<XapiStatement, 'stored'>;
 export declare type EagerXapiStatement = Omit<XapiStatement, 'stored'>;
+export declare type UXapiStatement = XapiStatement | EagerXapiStatement | SavedXapiStatement;
 export declare type LrsGateway = ReturnType<ReturnType<ReturnType<typeof lrsGateway>>>;
 export declare type LrsProvider = ReturnType<ReturnType<typeof lrsGateway>>;
 export declare const lrsGateway: <C extends string = "lrs">(initializer: Initializer<C>) => (configProvider: { [key in C]: {
@@ -82,7 +83,7 @@ export declare const lrsGateway: <C extends string = "lrs">(initializer: Initial
 }; }) => (authProvider: AuthProvider) => {
     putXapiStatements: (statements: Array<(Pick<XapiStatement, 'object' | 'verb' | 'context' | 'result'> & {
         id?: string;
-    }) | EagerXapiStatement>) => Promise<EagerXapiStatement[]>;
+    }) | UXapiStatement>) => Promise<EagerXapiStatement[]>;
     getXapiStatements: (params: {
         verb?: string | undefined;
         activity?: string | undefined;