@fluojs/discord 1.0.0-beta.2 → 1.0.0

package/README.ko.md

@@ -61,8 +61,9 @@ export class AppModule {}
 import { Inject } from '@fluojs/core';
 import { DiscordService } from '@fluojs/discord';
 
+@Inject(DiscordService)
 export class DeployNotifier {
-  constructor(@Inject(DiscordService) private readonly discord: DiscordService) {}
+  constructor(private readonly discord: DiscordService) {}
 
   async announce(version: string) {
     await this.discord.send({
@@ -95,6 +96,7 @@ Behavioral contract 메모:
 
 - `DiscordService.send(...)`는 전달 전에 `defaultThreadId`를 해석합니다.
 - 서비스는 모듈 bootstrap 시 transport를 초기화하고, factory가 소유한 리소스만 애플리케이션 shutdown 시 닫습니다.
+- 서비스가 shutdown 중이거나 이미 stopped 상태라면 cached transport를 재사용하지 않고 send를 거부합니다.
 - 빈 `defaultThreadId`와 `notifications.channel` 값은 trim 후 무시됩니다. notifications channel은 기본적으로 `discord`입니다.
 - 이 패키지는 절대로 `process.env`를 직접 읽지 않습니다. 모든 설정은 명시적인 옵션 또는 DI를 통해 들어와야 합니다.
 
@@ -163,7 +165,7 @@ bot 기반 REST 전달처럼 더 풍부한 API 연동이 필요하다면 export
 
 Behavioral contract 메모:
 
-- 내장 webhook transport는 `408`, `429`, `5xx` 같은 일시적 응답뿐 아니라 transport-level exception도 bounded exponential backoff로 재시도한 뒤 호출자에게 에러를 노출합니다.
+- 내장 webhook transport는 `408`, `429`, `5xx` 같은 일시적 응답뿐 아니라 transport-level exception도 bounded exponential backoff로 재시도한 뒤 호출자에게 에러를 노출합니다. 영구적인 upstream 응답은 재시도하지 않습니다.
 - 잘못되었거나 절대 URL이 아닌 `webhookUrl` 값은 전달 실패로 재시도하지 않고 즉시 `DiscordConfigurationError`로 거부됩니다.
 - 호출자에게 보이는 `DiscordTransportError` 메시지는 기본적으로 raw upstream response body를 포함하지 않습니다.
 

package/README.md

@@ -61,8 +61,9 @@ export class AppModule {}
 import { Inject } from '@fluojs/core';
 import { DiscordService } from '@fluojs/discord';
 
+@Inject(DiscordService)
 export class DeployNotifier {
-  constructor(@Inject(DiscordService) private readonly discord: DiscordService) {}
+  constructor(private readonly discord: DiscordService) {}
 
   async announce(version: string) {
     await this.discord.send({
@@ -95,6 +96,7 @@ Behavioral contract notes:
 
 - `DiscordService.send(...)` resolves `defaultThreadId` before delivery.
 - The service initializes the configured transport during module bootstrap and closes factory-owned resources during application shutdown.
+- Sends attempted while the service is shutting down or already stopped are rejected before reusing the cached transport.
 - Blank `defaultThreadId` and `notifications.channel` values are trimmed and ignored; the notifications channel defaults to `discord`.
 - The package never reads `process.env` directly. All configuration must enter through explicit options or DI.
 
@@ -163,7 +165,7 @@ For richer API integrations such as bot-backed REST delivery, implement the expo
 
 Behavioral contract notes:
 
-- The built-in webhook transport retries transient `408`, `429`, and `5xx` responses, and also retries transport-level exceptions, using bounded exponential backoff before surfacing an error.
+- The built-in webhook transport retries transient `408`, `429`, and `5xx` responses, and also retries transport-level exceptions, using bounded exponential backoff before surfacing an error. Permanent upstream responses are not retried.
 - Malformed or non-absolute `webhookUrl` values are rejected immediately as `DiscordConfigurationError` instead of being retried as delivery failures.
 - Caller-visible `DiscordTransportError` messages omit raw upstream response bodies by default.
 

package/dist/service.d.ts

@@ -71,6 +71,7 @@ export declare class DiscordService implements Discord, OnModuleInit, OnApplicat
      */
     sendNotification(notification: DiscordNotificationDispatchRequest, options?: DiscordSendOptions): Promise<DiscordSendResult>;
     private ensureTransport;
+    private assertReadyForSend;
     private normalizeMessage;
     private resolveNotificationThreadId;
     private renderNotification;

package/dist/service.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"service.d.ts","sourceRoot":"","sources":["../src/service.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,qBAAqB,EAAE,YAAY,EAAE,MAAM,iBAAiB,CAAC;AAK3E,OAAO,KAAK,EACV,OAAO,EACP,cAAc,EACd,kCAAkC,EAClC,sBAAsB,EAEtB,sBAAsB,EACtB,kBAAkB,EAClB,iBAAiB,EAIjB,8BAA8B,EAC/B,MAAM,YAAY,CAAC;AAqBpB;;;;;;;GAOG;AACH,qBACa,cAAe,YAAW,OAAO,EAAE,YAAY,EAAE,qBAAqB;IAKrE,OAAO,CAAC,QAAQ,CAAC,OAAO;IAJpC,OAAO,CAAC,cAAc,CAAmF;IACzG,OAAO,CAAC,iBAAiB,CAA+B;IACxD,OAAO,CAAC,gBAAgB,CAAwC;gBAEnC,OAAO,EAAE,8BAA8B;IAE9D,qBAAqB,IAAI,OAAO,CAAC,IAAI,CAAC;IAetC,YAAY,IAAI,OAAO,CAAC,IAAI,CAAC;IAiBnC;;;;OAIG;IACH,4BAA4B;IAW5B;;;;;;;;;;;;;;OAcG;IACG,IAAI,CAAC,OAAO,EAAE,cAAc,EAAE,OAAO,GAAE,kBAAuB,GAAG,OAAO,CAAC,iBAAiB,CAAC;IAuBjG;;;;;;;;;;;;OAYG;IACG,QAAQ,CAAC,QAAQ,EAAE,SAAS,cAAc,EAAE,EAAE,OAAO,GAAE,sBAA2B,GAAG,OAAO,CAAC,sBAAsB,CAAC;IA6B1H;;;;;;;;;;;;;;;;OAgBG;IACG,gBAAgB,CACpB,YAAY,EAAE,kCAAkC,EAChD,OAAO,GAAE,kBAAuB,GAC/B,OAAO,CAAC,iBAAiB,CAAC;YA6Bf,eAAe;IAe7B,OAAO,CAAC,gBAAgB;IAkBxB,OAAO,CAAC,2BAA2B;YAkBrB,kBAAkB;CAejC"}
+{"version":3,"file":"service.d.ts","sourceRoot":"","sources":["../src/service.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,qBAAqB,EAAE,YAAY,EAAE,MAAM,iBAAiB,CAAC;AAK3E,OAAO,KAAK,EACV,OAAO,EACP,cAAc,EACd,kCAAkC,EAClC,sBAAsB,EAEtB,sBAAsB,EACtB,kBAAkB,EAClB,iBAAiB,EAIjB,8BAA8B,EAC/B,MAAM,YAAY,CAAC;AAyBpB;;;;;;;GAOG;AACH,qBACa,cAAe,YAAW,OAAO,EAAE,YAAY,EAAE,qBAAqB;IAKrE,OAAO,CAAC,QAAQ,CAAC,OAAO;IAJpC,OAAO,CAAC,cAAc,CAAmF;IACzG,OAAO,CAAC,iBAAiB,CAA+B;IACxD,OAAO,CAAC,gBAAgB,CAAwC;gBAEnC,OAAO,EAAE,8BAA8B;IAE9D,qBAAqB,IAAI,OAAO,CAAC,IAAI,CAAC;IAetC,YAAY,IAAI,OAAO,CAAC,IAAI,CAAC;IAiBnC;;;;OAIG;IACH,4BAA4B;IAW5B;;;;;;;;;;;;;;OAcG;IACG,IAAI,CAAC,OAAO,EAAE,cAAc,EAAE,OAAO,GAAE,kBAAuB,GAAG,OAAO,CAAC,iBAAiB,CAAC;IAyBjG;;;;;;;;;;;;OAYG;IACG,QAAQ,CAAC,QAAQ,EAAE,SAAS,cAAc,EAAE,EAAE,OAAO,GAAE,sBAA2B,GAAG,OAAO,CAAC,sBAAsB,CAAC;IA6B1H;;;;;;;;;;;;;;;;OAgBG;IACG,gBAAgB,CACpB,YAAY,EAAE,kCAAkC,EAChD,OAAO,GAAE,kBAAuB,GAC/B,OAAO,CAAC,iBAAiB,CAAC;YAiCf,eAAe;IAe7B,OAAO,CAAC,kBAAkB;IAM1B,OAAO,CAAC,gBAAgB;IAkBxB,OAAO,CAAC,2BAA2B;YAkBrB,kBAAkB;CAejC"}

package/dist/service.js

@@ -5,7 +5,7 @@ function _toPrimitive(t, r) { if ("object" != typeof t || !t) return t; var e =
 function _setFunctionName(e, t, n) { "symbol" == typeof t && (t = (t = t.description) ? "[" + t + "]" : ""); try { Object.defineProperty(e, "name", { configurable: !0, value: n ? n + " " + t : t }); } catch (e) {} return e; }
 function _checkInRHS(e) { if (Object(e) !== e) throw TypeError("right-hand side of 'in' should be an object, got " + (null !== e ? typeof e : "null")); return e; }
 import { Inject } from '@fluojs/core';
-import { DiscordMessageValidationError } from './errors.js';
+import { DiscordMessageValidationError, DiscordTransportError } from './errors.js';
 import { createDiscordPlatformStatusSnapshot } from './status.js';
 import { DISCORD_OPTIONS } from './tokens.js';
 function createAbortError() {
@@ -13,6 +13,9 @@ function createAbortError() {
   error.name = 'AbortError';
   return error;
 }
+function createStoppedTransportError() {
+  return new DiscordTransportError('Discord transport is shutting down or already stopped.');
+}
 function normalizeOptionalString(value) {
   const trimmed = value?.trim();
   return trimmed && trimmed.length > 0 ? trimmed : undefined;
@@ -49,9 +52,11 @@ class DiscordService {
         await this.resolvedTransport.close();
       }
       this.lifecycleState = 'stopped';
-    } catch {
+    } catch (error) {
       this.lifecycleState = 'failed';
-      throw new Error('Discord transport failed to close cleanly.');
+      throw new Error('Discord transport failed to close cleanly.', {
+        cause: error
+      });
     }
   }
   async onModuleInit() {
@@ -62,9 +67,11 @@ class DiscordService {
         await transport.verify();
       }
       this.lifecycleState = 'ready';
-    } catch {
+    } catch (error) {
       this.lifecycleState = 'failed';
-      throw new Error('Discord transport failed to initialize.');
+      throw new Error('Discord transport failed to initialize.', {
+        cause: error
+      });
     }
   }
 
@@ -103,6 +110,7 @@ class DiscordService {
     if (options.signal?.aborted) {
       throw createAbortError();
     }
+    this.assertReadyForSend();
     const transport = await this.ensureTransport();
     const normalized = this.normalizeMessage(message);
     assertMessageContent(normalized);
@@ -176,6 +184,9 @@ class DiscordService {
    * ```
    */
   async sendNotification(notification, options = {}) {
+    if (options.signal?.aborted) {
+      throw createAbortError();
+    }
     const payload = notification.payload;
     const rendered = await this.renderNotification(notification);
     return this.send({
@@ -215,6 +226,11 @@ class DiscordService {
     }
     return this.transportPromise;
   }
+  assertReadyForSend() {
+    if (this.lifecycleState === 'stopping' || this.lifecycleState === 'stopped') {
+      throw createStoppedTransportError();
+    }
+  }
   normalizeMessage(message) {
     return {
       allowedMentions: message.allowedMentions,

package/dist/webhook.js

@@ -184,13 +184,13 @@ export function createDiscordWebhookTransport(options) {
           if (isAbortError(error) || context.signal?.aborted) {
             throw error;
           }
+          if (error instanceof DiscordTransportError) {
+            throw error;
+          }
           if (attempt < DEFAULT_RETRY_ATTEMPTS) {
             await waitForRetry(DEFAULT_RETRY_DELAY_MS * 2 ** (attempt - 1), context.signal);
             continue;
           }
-          if (error instanceof DiscordTransportError) {
-            throw error;
-          }
           throw new DiscordTransportError(createTransportFailureMessage(attempt));
         }
       }

package/package.json

@@ -9,7 +9,7 @@
     "portable",
     "fetch"
   ],
-  "version": "1.0.0-beta.2",
+  "version": "1.0.0",
   "private": false,
   "license": "MIT",
   "repository": {
@@ -36,10 +36,10 @@
     "dist"
   ],
   "dependencies": {
-    "@fluojs/core": "^1.0.0-beta.4",
-    "@fluojs/di": "^1.0.0-beta.6",
-    "@fluojs/notifications": "^1.0.0-beta.3",
-    "@fluojs/runtime": "^1.0.0-beta.11"
+    "@fluojs/core": "^1.0.0",
+    "@fluojs/di": "^1.0.0",
+    "@fluojs/notifications": "^1.0.0",
+    "@fluojs/runtime": "^1.0.0"
   },
   "devDependencies": {
     "vitest": "^3.2.4"