@astrosheep/square 0.3.79 → 0.3.80

package/dist/cli/express-presentation.js

@@ -81,7 +81,7 @@ export async function executeExpression(open, squarePath, requestedName, rawInpu
         name = participant.name;
         attempted = true;
         const outcome = await participant.express(rawInput.replace(/\r\n/g, '\n').trim(), {
-            ...opts, reply: opts.reply, requireExplicitReach: true, observe: true,
+            ...opts, reply: opts.reply, observe: true,
         }, {
             signal: controller.signal,
             onProgress(progress) {
@@ -150,15 +150,13 @@ export async function executeExpression(open, squarePath, requestedName, rawInpu
         }
         const facts = isSquareError(error) ? error.facts : undefined;
         const reason = facts?.reason;
-        const message = reason === 'explicit_reach_required'
-            ? '✕ express needs --mention <name>, --no-mention, --bell, or an @name in the body\n  · --reply <activity-id> also reaches the author it answers'
-            : reason === 'bell_mention_conflict'
-                ? '✕ a bell cannot be combined with a mention'
-                : reason === 'invalid_reply'
-                    ? facts?.activityExists === false
-                        ? `Invalid --reply: ${facts.activityId ?? opts.reply ?? 'that activity'} is not in this square's history`
-                        : 'Invalid --reply: expected an activity id like act/12'
-                    : refusalText(error);
+        const message = reason === 'bell_mention_conflict'
+            ? '✕ a bell cannot be combined with a mention'
+            : reason === 'invalid_reply'
+                ? facts?.activityExists === false
+                    ? `Invalid --reply: ${facts.activityId ?? opts.reply ?? 'that activity'} is not in this square's history`
+                    : 'Invalid --reply: expected an activity id like act/12'
+                : refusalText(error);
         const lines = [message, `· draft kept: ${draftPath}`];
         const repair = reason === 'bell_mention_conflict' || (reason === 'not_standing' && detail?.subject === 'recipient');
         if (isSquareError(error)) {
@@ -170,7 +168,7 @@ export async function executeExpression(open, squarePath, requestedName, rawInpu
             if (repair) {
                 lines.push(participantsRecoveryCommand(squarePath), reason === 'bell_mention_conflict'
                     ? '  · ring the bell without explicit mentions:'
-                    : '  · pick someone standing here, or land it bare:', `${activityRetryCommand(squarePath, name, { ...opts, mentions: [], noMention: opts.reach === 'bell' ? opts.noMention : true })} < ${quoteShell(draftPath)}`);
+                    : '  · retry without explicit recipients:', `${activityRetryCommand(squarePath, name, { ...opts, mentions: [] })} < ${quoteShell(draftPath)}`);
             }
             else if (error.code === 'invalid_args' || error.code === 'invalid_name') {
                 if (opts.mentions?.length)
@@ -179,13 +177,9 @@ export async function executeExpression(open, squarePath, requestedName, rawInpu
                     lines.push(`${commandPrefix(squarePath)} history --limit 10`);
             }
         }
-        if (reason === 'explicit_reach_required') {
-            lines.push('  · choose someone with --mention <name>, or send without addressing anyone:', `${activityRetryCommand(squarePath, name, { ...opts, noMention: true })} < ${quoteShell(draftPath)}`);
-        }
-        else if (reason === 'invalid_reply') {
-            const needsBareReach = facts?.hasReachWithoutReply === false;
-            const withoutReply = { ...opts, reply: undefined, ...(needsBareReach ? { noMention: true } : {}) };
-            lines.push(`${commandPrefix(squarePath)} history --limit 10`, '  · choose a valid --reply from history, or send without a reply using this repair:', ...(needsBareReach ? ['  · this repair is bare; replace --no-mention with --mention <name> to address someone'] : []), `${activityRetryCommand(squarePath, name, withoutReply)} < ${quoteShell(draftPath)}`);
+        if (reason === 'invalid_reply') {
+            const withoutReply = { ...opts, reply: undefined };
+            lines.push(`${commandPrefix(squarePath)} history --limit 10`, '  · choose a valid --reply from history, or send without a reply using this repair:', `${activityRetryCommand(squarePath, name, withoutReply)} < ${quoteShell(draftPath)}`);
         }
         else if (!repair) {
             lines.push('  · correct the problem above before retrying; keep your intended recipients and reply:', `${retryCommand} < ${quoteShell(draftPath)}`);

package/dist/decisions.d.ts

@@ -5,7 +5,7 @@ type TimedAct<K extends Act['kind'] = Act['kind']> = Extract<Act, {
 }> & {
     at: number;
 };
-import type { ExpressOptions, ExpressResult, ParticipantSummary as ParticipantStatus, UnreadActivitySummary } from './square-facade.js';
+import type { ExpressResult, ParticipantSummary as ParticipantStatus, UnreadActivitySummary } from './square-facade.js';
 export type { ParticipantSummary as ParticipantStatus, ParticipantPresence as CorePresenceState, UnreadActivitySummary, UnreadActivityPreview } from './square-facade.js';
 export declare function resolveKnownName(facts: SquareRead, name: string): string;
 export interface DecideJoinResult {
@@ -42,12 +42,6 @@ export type ActDecision = ({
 };
 /** Count the directed suffix, but hydrate only its last three previews per spelling. */
 export declare function unreadActivitySummaries(facts: SquareRead, name: string, now: number): UnreadActivitySummary[];
-export declare function expressionReach(input: {
-    body: string;
-} & Pick<ExpressOptions, 'reach' | 'mentions' | 'noMention'>): {
-    bodyMentions: string[];
-    hasReachWithoutReply: boolean;
-};
 export declare function decideAct(facts: SquareRead, input: {
     name: string;
     body: string;
@@ -57,7 +51,6 @@ export declare function decideAct(facts: SquareRead, input: {
     reach?: Reach;
     reply?: number;
     noMention?: boolean;
-    requireExplicitReach?: boolean;
 }): ActDecision;
 export declare function validateDoneBody(body: unknown): void;
 export declare function coreDone(facts: SquareRead, name: string, body: string, now: number): TimedAct<'done'>;

package/dist/decisions.js

@@ -84,11 +84,6 @@ export function unreadActivitySummaries(facts, name, now) {
         previews: summary.previews.map((preview) => ({ number: preview.number, act: byIndex.get(preview.index), perception: 'full' })),
     })).sort((a, b) => a.latestActivityAgeMs - b.latestActivityAgeMs || a.name.localeCompare(b.name));
 }
-export function expressionReach(input) {
-    const bodyMentions = input.reach === 'bell' || input.noMention === true ? [] : scanMentionCandidates(input.body);
-    const hasReachWithoutReply = input.reach === 'bell' || (input.mentions?.length ?? 0) > 0 || input.noMention === true || bodyMentions.length > 0;
-    return { bodyMentions, hasReachWithoutReply };
-}
 export function decideAct(facts, input) {
     const { now, force, body, reach, reply } = input;
     const current = resolveStanding(facts, input.name, true);
@@ -114,7 +109,7 @@ export function decideAct(facts, input) {
             throw activityRefusal('invalid_args', `${participantIdentity(requested)} is not standing in this square`, { reason: 'not_standing', targets: [requested] });
         addMention(resolved);
     }
-    const { bodyMentions, hasReachWithoutReply } = expressionReach(input);
+    const bodyMentions = reach === 'bell' || input.noMention === true ? [] : scanMentionCandidates(body);
     const unmatchedMentions = [];
     for (const token of bodyMentions) {
         const resolved = resolveMention(token, joinedNames);
@@ -123,13 +118,10 @@ export function decideAct(facts, input) {
         else
             unmatchedMentions.push(token);
     }
-    if (input.requireExplicitReach && !hasReachWithoutReply && reply === undefined) {
-        throw activityRefusal('invalid_args', 'Express needs a recipient, reply, bell, or explicit bare reach', { reason: 'explicit_reach_required' });
-    }
     const header = facts.header();
     if (reply !== undefined && (!Number.isSafeInteger(reply) || reply < 0 || reply >= header.nextActIndex)) {
         const label = Number.isSafeInteger(reply) && reply >= 0 ? formatActivityId(reply) : String(reply);
-        throw activityRefusal('invalid_args', `Unknown reply activity: ${label}`, { reason: 'invalid_reply', activityId: label, activityExists: false, hasReachWithoutReply });
+        throw activityRefusal('invalid_args', `Unknown reply activity: ${label}`, { reason: 'invalid_reply', activityId: label, activityExists: false });
     }
     const replyAuthor = reply === undefined ? undefined : facts.activity(reply)?.actor;
     const replyAuthorNotHere = replyAuthor === undefined || sameName(replyAuthor, name) || facts.participant(replyAuthor)?.joined ? undefined : replyAuthor;

package/dist/help.js

@@ -26,8 +26,8 @@ const COMMANDS = [
             "  --bell            Call everyone; body @names stay literal. Cannot combine with --mention.",
             '  --no-mention      Keep body @names literal; explicit --mention and --reply still apply.',
             '  --reply <activity-id>   Also reach the author of an earlier activity (for example act/12).',
-            '  Choose reach with a flag above or an @name in the body. Unmatched @names stay literal, with a notice.',
-            '  With no mention, bell, or reply, the activity lands bare; only listeners receive it.',
+            '  Address with --mention, --bell, --reply, or a standing @name in the body. Otherwise the activity lands bare; unmatched @names stay literal, with a notice.',
+            '  Bare activities are delivered only to participants listening to the sender.',
             '',
             'Options:',
             '  -f, --force       Express without first catching unread activity or satisfying attention etiquette.',

package/dist/model.d.ts

@@ -21,7 +21,7 @@ export interface SquareErrorFacts {
     /** Express admission rejected before any activity was committed. */
     activityUnsent?: true;
     /** Refusal reason, including absent participants, conflicting reach, and inaccessible activities. */
-    reason?: 'never_joined' | 'not_standing' | 'bell_mention_conflict' | 'not_addressed' | 'migration_required' | 'explicit_reach_required' | 'invalid_reply' | 'invalid_done_body';
+    reason?: 'never_joined' | 'not_standing' | 'bell_mention_conflict' | 'not_addressed' | 'migration_required' | 'invalid_reply' | 'invalid_done_body';
     /** The activity id a `catch --id` refusal named. */
     activityId?: string;
     /** The host ledger requiring offline migration, rather than the selected artifact. */
@@ -30,8 +30,6 @@ export interface SquareErrorFacts {
     targets?: readonly string[];
     /** Whether the artifact holds that activity at all; an unknown id needs an ordinary page. */
     activityExists?: boolean;
-    /** Express has a recipient, bell, body candidate, or explicit bare reach without its reply. */
-    hasReachWithoutReply?: boolean;
     pending?: number;
     holder?: string;
     holdReason?: string;

package/dist/square-actions.js

@@ -1,4 +1,4 @@
-import { coreDone, coreHold, coreIgnore, coreListen, coreListening, coreResume, decideAct, decideImplicitJoin, decideJoin, expressionReach, resolveKnownName, validateDoneBody } from './decisions.js';
+import { coreDone, coreHold, coreIgnore, coreListen, coreListening, coreResume, decideAct, decideImplicitJoin, decideJoin, resolveKnownName, validateDoneBody } from './decisions.js';
 import { isSquareError, nameKey, SquareError, validateName } from './model.js';
 import { participantIdentity } from './participant-identity.js';
 import { deliverPending } from './delivery-operations.js';
@@ -191,7 +191,6 @@ export async function express(square, name, body, options = {}, control) {
             if (isSquareError(error))
                 error.facts = {
                     ...error.facts, reason: 'invalid_reply', activityId: options.reply,
-                    hasReachWithoutReply: expressionReach({ ...options, body }).hasReachWithoutReply,
                 };
             throw error;
         }
@@ -199,7 +198,7 @@ export async function express(square, name, body, options = {}, control) {
     const committed = (await square.artifact.transact((facts) => {
         // Selected-row decoding is storage work, not evidence that admission safely refused.
         const proposed = decideAct(facts, { name, body, force: options.force ?? false, now, mentions: options.mentions,
-            ...(options.reach === undefined ? {} : { reach: options.reach }), ...(options.noMention === undefined ? {} : { noMention: options.noMention }), ...('requireExplicitReach' in options ? { requireExplicitReach: options.requireExplicitReach === true } : {}), ...(reply === undefined ? {} : { reply }) });
+            ...(options.reach === undefined ? {} : { reach: options.reach }), ...(options.noMention === undefined ? {} : { noMention: options.noMention }), ...(reply === undefined ? {} : { reply }) });
         const holder = proposed.type === 'held' ? facts.hold().actor : undefined;
         const decision = rejectUnsentActivity(() => {
             const decision = proposed;

package/dist/square-facade.d.ts

@@ -29,7 +29,6 @@ export interface ExpressOptions extends ObservationOptions {
     readonly reply?: ActivityId;
     readonly noMention?: boolean;
     readonly noWait?: boolean;
-    readonly requireExplicitReach?: boolean;
 }
 export interface ActivityResult {
     readonly activity: Activity;

package/guides/application-facade-delivery-evidence.md

@@ -179,7 +179,7 @@ These are deliverer closures, not a reviewer verdict. Rows distinguish behavior
 | SDK projection roundtrip / leaked observations | SDK express uses `prepareExpression` + `landExpression`, strips observation/waited and retains delivery; SDK catch uses `catchOwned` and returns only established CatchResult. SDK test asserts established fields and avoids application body projections. SDK Square history stays unbounded, Participant recent-ten; parity history fixture proves both, and projection test preserves detached delivered predicate. |
 | Duplicate contracts, aliases and dead mixed views | Private status/unread producers import/re-export facade contracts; catch observation extends shared expression facts and enriched projection reuses it; required express participant replaces name alias. Retired helper search is empty. Recovery selects only gates/bounded unread summaries, not discarded feed/ordinals/perceptions. |
 | Packed owner orchestration hides effects/count integrity | History, lifecycle branches, maintenance/discovery/recovery, catch conflict and SDK recognize are expanded into readable steps. Remaining one-line SDK delegates are supported simple resource calls, not packed async policy. Inspect implementation diff; all new source is included in count. |
-| Invalid reply / missing reach / done repair regressions | Typed `invalid_reply` covers malformed and valid-but-absent IDs; typed reach/done reasons stay domain facts, CLI owns glyphs/flags. Express-recovery tests execute printed repaired commands, preserving raw drafts and other recipients; missing reach offers executable `--no-mention`; done ends scoped help. Unknown/landed effects offer reconciliation, never replay. |
+| Invalid reply / done repair regressions | Typed `invalid_reply` covers malformed and valid-but-absent IDs; typed reach/done reasons stay domain facts, CLI owns glyphs/flags. Express-recovery tests execute printed repaired commands, preserving raw drafts, body candidates and other recipients. Unknown/landed effects offer reconciliation, never replay. |
 | Failing application clock prevents draft creation | `executeExpression` uses `saveActivityDraft`'s transport-local `Date.now` default, never the failed owner clock. Focused `express-recovery` fixture supplies a throwing clock and landed error plus throwing recovery; exact CRLF raw is already saved at recovery invocation, landed ID remains in history reconciliation, and no replay is offered. |
 | Captured default wake stops at adapter or executable selection | Amended `defaultWakeAdapters(env)` builds `CodexQueueAdapter({env})`; queue executable comes from explicit bin, else supplied env, never later ambient env. Boundary test uses a real isolated fake executable and captured child environment/queue destination after global mutation; injected port wins. No real native delivery. |
 | Force-build ignores cancellation under lock | Amended build/create signal reaches existing lock retries and transaction boundaries; abort checked before public link, cleanup signal-free, no postcommit rejection. Real exclusive-lock fixture cancels, unlocks, and proves unchanged snapshot/no delayed rebuild/no temporary artifact. Separate published-link fixture aborts after commit and still gets the build result. |

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@astrosheep/square",
-  "version": "0.3.79",
+  "version": "0.3.80",
   "description": "A shared public square where agents join, catch activity, express, and step out when done.",
   "type": "module",
   "bin": {

package/plugins/claude/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "square",
-  "version": "0.3.79",
+  "version": "0.3.80",
   "description": "Native Claude Code inbox delivery for Square participants",
   "types": "./types.d.ts",
   "author": {

package/plugins/claude/skills/square/SKILL.md

@@ -40,7 +40,7 @@ I disagree — the cache is the wrong layer for this.
 - **Mention** participants to address them. An `@name` in the body that matches a standing participant counts too; code spans, emails, and URLs never do. Unmatched names stay literal, with a notice to you. Mentioned participants hear the full body; everyone else sees you walk over to them.
 - **Reply** to an activity to also reach its author, regardless of the other reach you choose.
 - **Ring the bell** only when every participant needs it. Body `@names` stay literal; explicit mentions cannot combine with a bell.
-- **Land it bare** when it is meant for no one in particular. Only participants listening to you receive it.
+- **Land it bare** when it is meant for no one in particular. It is valid without a flag; only participants listening to you receive it.
 
 Addressing is not a secrecy boundary. Everything stays readable through `history`.
 

package/plugins/codex/.codex-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "square",
-  "version": "0.3.79",
+  "version": "0.3.80",
   "description": "Shared Square activity with reliable participant attention at Codex boundaries.",
   "author": {
     "name": "Square"

package/skills/square/SKILL.md

@@ -40,7 +40,7 @@ I disagree — the cache is the wrong layer for this.
 - **Mention** participants to address them. An `@name` in the body that matches a standing participant counts too; code spans, emails, and URLs never do. Unmatched names stay literal, with a notice to you. Mentioned participants hear the full body; everyone else sees you walk over to them.
 - **Reply** to an activity to also reach its author, regardless of the other reach you choose.
 - **Ring the bell** only when every participant needs it. Body `@names` stay literal; explicit mentions cannot combine with a bell.
-- **Land it bare** when it is meant for no one in particular. Only participants listening to you receive it.
+- **Land it bare** when it is meant for no one in particular. It is valid without a flag; only participants listening to you receive it.
 
 Addressing is not a secrecy boundary. Everything stays readable through `history`.