firstly 0.9.2 → 0.9.4

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # firstly
 
+## 0.9.4
+
+### Patch Changes
+
+- [#352](https://github.com/jycouet/firstly/pull/352) [`53ec5e1`](https://github.com/jycouet/firstly/commit/53ec5e12f744ed207b9fce6fa80f3bc9b4918a92) Thanks [@jycouet](https://github.com/jycouet)! - feedback: `getIssues` returned nothing when `milestones.labels_filters` was unset (GitHub treats `labels: []` as "match nothing"); the filter is now omitted in that case.
+
+## 0.9.3
+
+### Patch Changes
+
+- [#349](https://github.com/jycouet/firstly/pull/349) [`b285731`](https://github.com/jycouet/firstly/commit/b285731fe8e6cd41e5a257a044524486319e5544) Thanks [@jycouet](https://github.com/jycouet)! - feedback: `getIssues` now returns `labels` (names), `createdAt` and `updatedAt` per issue; `createIssue` accepts an optional `labels: string[]` applied on top of `create_label`.
+
+- [#351](https://github.com/jycouet/firstly/pull/351) [`9701c40`](https://github.com/jycouet/firstly/commit/9701c404c7c2e6e2f94a7e3eab536b6a55d0c9c2) Thanks [@jycouet](https://github.com/jycouet)! - `withShortTermCache`: mutations (POST writes, PUT, PATCH, DELETE) now clear the cache once they settle, so a follow-up refresh no longer serves the pre-mutation read.
+
 ## 0.9.2
 
 ### Patch Changes

package/esm/core/httpClientStack.d.ts

@@ -28,7 +28,8 @@ export type ShortTermCacheOptions = {
  * Middleware that dedupes identical read requests within a short TTL - a tiny
  * client-side cache. Two components mounting and issuing the same query result in
  * ONE network call; each caller gets its own `Response` clone. Failed requests are
- * evicted immediately so the next call retries.
+ * evicted immediately so the next call retries. Any mutation (non-GET that isn't a
+ * cached read) clears the cache once it settles, so a follow-up refresh sees fresh data.
  *
  * ```ts
  * remult.apiClient.httpClient = stackHttpClient(withShortTermCache({ ttlMs: 2000 }))

package/esm/core/httpClientStack.js

@@ -26,7 +26,8 @@ const defaultShouldCache = (_input, init, key) => {
  * Middleware that dedupes identical read requests within a short TTL - a tiny
  * client-side cache. Two components mounting and issuing the same query result in
  * ONE network call; each caller gets its own `Response` clone. Failed requests are
- * evicted immediately so the next call retries.
+ * evicted immediately so the next call retries. Any mutation (non-GET that isn't a
+ * cached read) clears the cache once it settles, so a follow-up refresh sees fresh data.
  *
  * ```ts
  * remult.apiClient.httpClient = stackHttpClient(withShortTermCache({ ttlMs: 2000 }))
@@ -48,7 +49,18 @@ export function withShortTermCache(opts = {}) {
             cacheKey = `${cacheKey}:${bodyStr}`;
         }
         if (!shouldCache(input, init, cacheKey)) {
-            return next(input, init);
+            const method = init?.method?.toUpperCase() ?? 'GET';
+            if (method === 'GET')
+                return next(input, init);
+            // Mutation: drop cached reads once it settles (not before, so a read racing the
+            // write can't re-cache pre-mutation data). Cleared on error too: the server may
+            // have partially mutated.
+            try {
+                return await next(input, init);
+            }
+            finally {
+                cache.clear();
+            }
         }
         const now = Date.now();
         const cached = cache.get(cacheKey);

package/esm/feedback/FeedbackController.d.ts

@@ -9,6 +9,10 @@ export declare class FeedbackController {
         number: number;
         titleHTML: string;
         state: string;
+        createdAt: string;
+        updatedAt: string;
+        /** GitHub label names. */
+        labels: string[];
         highlight: boolean;
     }[]>;
     static getIssue(issueNumber: number): Promise<{
@@ -27,7 +31,9 @@ export declare class FeedbackController {
     }>;
     static createIssue(milestoneId: string, title: string, body: string, metadata: {
         page: string;
-    }): Promise<{
+    }, 
+    /** Extra label names applied on top of `create_label` (unknown names are ignored). */
+    labels?: string[]): Promise<{
         id: string;
         number: number;
     }>;

package/esm/feedback/FeedbackController.js

@@ -121,6 +121,8 @@ export class FeedbackController {
                 number
                 titleHTML
                 state
+                createdAt
+                updatedAt
                 labels(first:10){
                   nodes {
                     name
@@ -136,20 +138,27 @@ export class FeedbackController {
             owner: remult.context.feedbackOptions.repo.owner,
             milestoneNumber,
             filters: {
-                labels: remult.context.feedbackOptions.milestones?.labels_filters ?? [],
+                // GitHub treats `labels: []` as "match nothing", so omit the key when unset.
+                labels: remult.context.feedbackOptions.milestones?.labels_filters?.length
+                    ? remult.context.feedbackOptions.milestones.labels_filters
+                    : undefined,
                 states: [issueState],
             },
             issueOrder,
         });
         return data.repository.milestone.issues.nodes.map((issue) => {
+            const labels = issue.labels.nodes.map((label) => label.name);
             const hasWaitingForAnswerLabel = remult.context.feedbackOptions.highlight_label
-                ? issue.labels.nodes.some((label) => label.name.includes(remult.context.feedbackOptions.highlight_label))
+                ? labels.some((name) => name.includes(remult.context.feedbackOptions.highlight_label))
                 : false;
             return {
                 id: issue.id,
                 number: issue.number,
                 titleHTML: issue.titleHTML,
                 state: issue.state,
+                createdAt: issue.createdAt,
+                updatedAt: issue.updatedAt,
+                labels,
                 highlight: hasWaitingForAnswerLabel,
             };
         });
@@ -231,7 +240,9 @@ repository(name: $repository, owner: $owner) {
         };
         return toRet;
     }
-    static async createIssue(milestoneId, title, body, metadata) {
+    static async createIssue(milestoneId, title, body, metadata, 
+    /** Extra label names applied on top of `create_label` (unknown names are ignored). */
+    labels = []) {
         const repoInfo = await getGitHub(`query RepoInfo(
         $repository: String!
         $owner: String!
@@ -250,7 +261,8 @@ repository(name: $repository, owner: $owner) {
             owner: remult.context.feedbackOptions.repo.owner,
         });
         const repoInfoData = repoInfo.repository;
-        const create_label = repoInfoData.labels.nodes.find((c) => c.name === remult.context.feedbackOptions.create_label);
+        const wanted = new Set([remult.context.feedbackOptions.create_label, ...labels]);
+        const labelIds = repoInfoData.labels.nodes.filter((c) => wanted.has(c.name)).map((c) => c.id);
         const newIssue = await getGitHub(`mutation CreateIssue($input: CreateIssueInput!) {
       createIssue(input: $input) {
         issue {
@@ -263,7 +275,7 @@ repository(name: $repository, owner: $owner) {
             input: {
                 repositoryId: repoInfoData.id,
                 milestoneId: milestoneId,
-                labelIds: [create_label?.id],
+                labelIds,
                 title: title ?? 'New Feedback (wo title...)',
                 body: body,
             },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "firstly",
-  "version": "0.9.2",
+  "version": "0.9.4",
   "type": "module",
   "description": "Firstly, an opinionated Remult setup!",
   "funding": "https://github.com/sponsors/jycouet",