ag-common 0.0.746 → 0.0.748

package/dist/api/helpers/dynamo/delete.d.ts

@@ -6,8 +6,8 @@ export declare const batchDelete: (params: {
     opt?: {
         /** default 20 */
         batchSize?: number;
-        /** option to always retry on 429 until done. default false */
-        alwaysRetry?: boolean;
+        /** option to control retry behavior: undefined = 3 retries, null = infinite. default 3 */
+        maxRetries?: number | null;
     };
 }) => Promise<DynamoDBResult<void>>;
 export declare const wipeTable: (tableName: string) => Promise<DynamoDBResult<void>>;

package/dist/api/helpers/dynamo/delete.js

@@ -26,7 +26,7 @@ const get_1 = require("./get");
 const batchDelete = (params) => __awaiter(void 0, void 0, void 0, function* () {
     var _a;
     try {
-        const { batchSize = 20, alwaysRetry = false } = (_a = params.opt) !== null && _a !== void 0 ? _a : {};
+        const { batchSize = 20, maxRetries } = (_a = params.opt) !== null && _a !== void 0 ? _a : {};
         const chunked = (0, array_1.chunk)(params.keys, batchSize);
         let processed = 0;
         yield (0, async_1.asyncForEach)(chunked, (chunk) => __awaiter(void 0, void 0, void 0, function* () {
@@ -38,7 +38,7 @@ const batchDelete = (params) => __awaiter(void 0, void 0, void 0, function* () {
                 },
             };
             yield (0, withRetry_1.withRetry)(() => _1.dynamoDb.send(new lib_dynamodb_1.BatchWriteCommand(batchDeleteParams)), `batchdelete ${processed}/${params.keys.length}. size=${batchSize}`, {
-                maxRetries: alwaysRetry ? null : undefined,
+                maxRetries: maxRetries === undefined ? 3 : maxRetries,
             });
             processed += chunk.length;
         }));
@@ -54,7 +54,7 @@ const wipeTable = (tableName) => __awaiter(void 0, void 0, void 0, function* ()
     try {
         const generator = (0, get_1.scanWithGenerator)(tableName, {
             BATCH_SIZE: 100, // Process in chunks of 100 items
-            alwaysRetry: true, // Always retry on 429 since we want to ensure complete deletion
+            maxRetries: null, // Always retry on 429 since we want to ensure complete deletion
         });
         try {
             // Process each batch of items
@@ -69,7 +69,7 @@ const wipeTable = (tableName) => __awaiter(void 0, void 0, void 0, function* ()
                         keys: pks,
                         pkName: 'PK',
                         opt: {
-                            alwaysRetry: true, // Always retry on 429 since we want to ensure complete deletion
+                            maxRetries: null, // Always retry on 429 since we want to ensure complete deletion
                         },
                     });
                     if ('error' in result) {

package/dist/api/helpers/dynamo/get.js

@@ -66,7 +66,7 @@ const executeQuery = (params, startKey) => __awaiter(void 0, void 0, void 0, fun
         ExpressionAttributeValues: Object.assign(Object.assign({}, eav), params.filter.attrValues),
     }))));
     return (0, withRetry_1.withRetry)(() => _1.dynamoDb.send(new lib_dynamodb_1.QueryCommand(queryParams)), 'queryDynamo', {
-        maxRetries: params.alwaysRetry ? null : undefined,
+        maxRetries: params.maxRetries === undefined ? 3 : params.maxRetries,
     });
 });
 /**
@@ -89,7 +89,7 @@ const executeScan = (tableName, options, exclusiveStartKey) => __awaiter(void 0,
             .join(', '),
     })), { ExclusiveStartKey: exclusiveStartKey });
     return (0, withRetry_1.withRetry)(() => _1.dynamoDb.send(new lib_dynamodb_1.ScanCommand(scanParams)), `scan. already seen=${exclusiveStartKey ? 'some' : '0'} items`, {
-        maxRetries: (options === null || options === void 0 ? void 0 : options.alwaysRetry) ? null : undefined,
+        maxRetries: (options === null || options === void 0 ? void 0 : options.maxRetries) === undefined ? 3 : options.maxRetries,
     });
 });
 const getItemsDynamo = (params) => __awaiter(void 0, void 0, void 0, function* () {

package/dist/api/helpers/dynamo/index.d.ts

@@ -23,7 +23,7 @@ export declare const getItemDynamo: <T>(params: {
 export declare const putDynamo: <T extends Record<string, unknown>>(item: T, tableName: string, opt?: {
     pkName?: string;
 }) => Promise<import("./types").DynamoDBResult<void>>, batchWrite: <T extends Record<string, unknown>>(tableName: string, items: T[], opt?: {
-    alwaysRetry?: boolean;
+    maxRetries?: number | null;
     batchSize?: number;
 }) => Promise<import("./types").DynamoDBResult<void>>, getDynamoUpdates: <T extends Record<string, unknown>>(item: T, options?: {
     excludeKeys?: string[];
@@ -39,7 +39,7 @@ export declare const batchDelete: (params: {
     pkName: string;
     opt?: {
         batchSize?: number;
-        alwaysRetry?: boolean;
+        maxRetries?: number | null;
     };
 }) => Promise<import("./types").DynamoDBResult<void>>, wipeTable: (tableName: string) => Promise<import("./types").DynamoDBResult<void>>;
 export * from './types';

package/dist/api/helpers/dynamo/set.d.ts

@@ -3,8 +3,8 @@ export declare const putDynamo: <T extends Record<string, unknown>>(item: T, tab
     pkName?: string;
 }) => Promise<DynamoDBResult<void>>;
 export declare const batchWrite: <T extends Record<string, unknown>>(tableName: string, items: T[], opt?: {
-    /** option to always retry on 429 until done */
-    alwaysRetry?: boolean;
+    /** option to control retry behavior: undefined = 3 retries, null = infinite */
+    maxRetries?: number | null;
     /** default 20 */
     batchSize?: number;
 }) => Promise<DynamoDBResult<void>>;

package/dist/api/helpers/dynamo/set.js

@@ -40,7 +40,7 @@ const batchWrite = (tableName, items, opt) => __awaiter(void 0, void 0, void 0,
                 },
             };
             yield (0, withRetry_1.withRetry)(() => _1.dynamoDb.send(new lib_dynamodb_1.BatchWriteCommand(batchWriteParams)), `batchwrite ${processed}/${items.length}. size=${batchSize}`, {
-                maxRetries: (opt === null || opt === void 0 ? void 0 : opt.alwaysRetry) ? null : undefined,
+                maxRetries: (opt === null || opt === void 0 ? void 0 : opt.maxRetries) === undefined ? 3 : opt.maxRetries,
             });
             processed += chunk.length;
         }));

package/dist/api/helpers/dynamo/types.d.ts

@@ -23,7 +23,7 @@ export interface ScanOptions {
     filter?: DynamoFilter;
     requiredAttributeList?: string[];
     indexName?: string;
-    alwaysRetry?: boolean;
+    maxRetries?: number | null;
 }
 export interface DynamoQueryParams {
     tableName: string;
@@ -37,7 +37,7 @@ export interface DynamoQueryParams {
     limit?: number;
     filter?: DynamoFilter;
     sortAscending?: boolean;
-    alwaysRetry?: boolean;
+    maxRetries?: number | null;
 }
 export interface DynamoBatchQueryParams {
     tableName: string;

package/dist/api/helpers/s3.js

@@ -128,7 +128,13 @@ function getS3Objects(_a) {
 }
 const putS3Object = (_a) => __awaiter(void 0, [_a], void 0, function* ({ Body, Bucket, Key, ContentType, CacheControl = 'public, max-age=300', }) {
     try {
-        yield s3.send(new client_s3_1.PutObjectCommand({ Body, Bucket, Key, ContentType, CacheControl }));
+        const r = yield s3.send(new client_s3_1.PutObjectCommand({ Body, Bucket, Key, ContentType, CacheControl }));
+        const code = r.$metadata.httpStatusCode;
+        if (code !== 200) {
+            return {
+                error: `failed to upload file to s3. code=${code}. ${JSON.stringify(r.$metadata)}`,
+            };
+        }
         return {};
     }
     catch (e) {

package/dist/ui/components/LineChart/Base.js

@@ -50,16 +50,12 @@ const LineChart = (p) => {
     if (!tt && xTime) {
         tt = dateHelpers_1.timeTooltipTitle;
     }
-    if (!tt) {
-        tt = (s) => s.toString();
-    }
+    tt !== null && tt !== void 0 ? tt : (tt = (s) => s.toString());
     let lt = p.legendTitle;
     if (!lt && xTime) {
         lt = dateHelpers_1.timeLegendTitle;
     }
-    if (!lt) {
-        lt = (s) => s.toString();
-    }
+    lt !== null && lt !== void 0 ? lt : (lt = (s) => s.toString());
     const lt2 = lt;
     const tt2 = tt;
     const legendItems = (0, getLegendItems_1.getLegendItems)(Object.assign(Object.assign({ colours: p.colours, data: p.data, tt: tt2, lt: lt2 }, (_a = UT.pos) === null || _a === void 0 ? void 0 : _a.data), { fixed: false }));

package/dist/ui/components/LineChart/LegendY.js

@@ -33,12 +33,19 @@ const LegendY = ({ data, style, }) => {
     }
     items.push(maxY);
     const ch = maxY.toString().length;
+    const roundNumber = (num) => {
+        // Round to 2 decimal places and remove trailing zeros
+        const rounded = Math.round(num * 100) / 100;
+        return rounded % 1 === 0
+            ? rounded.toString()
+            : rounded.toFixed(2).replace(/\.?0+$/, '');
+    };
     return (react_1.default.createElement(Base, { style: Object.assign(Object.assign({}, style), { maxWidth: `${ch}ch`, paddingRight: '1ch' }) },
         react_1.default.createElement(Numbers, null, items.map((i, i2) => (react_1.default.createElement("span", { 
             // eslint-disable-next-line react/no-array-index-key
             key: i + i2, style: {
                 color: style.color,
                 backgroundColor: style.backgroundColor,
-            } }, i))))));
+            } }, roundNumber(i)))))));
 };
 exports.LegendY = LegendY;

package/dist/ui/components/LineChart/TooltipContent.js

@@ -34,18 +34,25 @@ const Total = styled_1.default.span `
 const TooltipContent = (p) => {
     var _a, _b;
     const name = p.tt((_b = (_a = p.selectedXs) === null || _a === void 0 ? void 0 : _a[0].x) !== null && _b !== void 0 ? _b : 0);
+    const roundNumber = (num) => {
+        // Round to 2 decimal places and remove trailing zeros
+        const rounded = Math.round(num * 100) / 100;
+        return rounded % 1 === 0
+            ? rounded.toString()
+            : rounded.toFixed(2).replace(/\.?0+$/, '');
+    };
     return (react_1.default.createElement(Base, { style: Object.assign(Object.assign({}, p.style), { border: `solid 1px ${p.style.borderColor}` }) },
         react_1.default.createElement(Title, null, name),
         react_1.default.createElement(Row, null,
             react_1.default.createElement("span", null, "total"),
-            react_1.default.createElement(Total, null, p.legendItems.total)),
+            react_1.default.createElement(Total, null, roundNumber(p.legendItems.total))),
         p.legendItems.part.map((v) => (react_1.default.createElement(Row, { key: v.name + v.y, style: { color: v.colour } },
             react_1.default.createElement(ItemTitle, null, v.name),
-            react_1.default.createElement(Total, null, v.y)))),
+            react_1.default.createElement(Total, null, roundNumber(v.y))))),
         p.legendItems.rest.length > 0 && (react_1.default.createElement(Row, null,
             react_1.default.createElement("span", null,
                 p.legendItems.rest.length,
                 " more"),
-            react_1.default.createElement(Total, null, p.legendItems.restTotal)))));
+            react_1.default.createElement(Total, null, roundNumber(p.legendItems.restTotal))))));
 };
 exports.TooltipContent = TooltipContent;

package/package.json

@@ -1,5 +1,5 @@
 {
-  "version": "0.0.746",
+  "version": "0.0.748",
   "name": "ag-common",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",