• .claude/skills/javascript/SKILL.md

    From Rob Swindell (on Debian Linux)@1:103/705 to Git commit to main/sbbs/master on Tue Jul 28 02:32:26 2026
    https://gitlab.synchro.net/main/sbbs/-/commit/74911bb1379078e52a52f4ca
    Modified Files:
    .claude/skills/javascript/SKILL.md
    Log Message:
    javascript skill: document the ini's "key : value" string literals

    A key separated from its value by a COLON rather than an equals sign makes
    the value a C string literal: xpdev/ini_file.c's key_name() runs it through c_unescape_str(). So an .ini can carry Ctrl-A attribute codes, CP437 bytes
    and control characters directly, spelled as ctrl/text.dat spells them, and
    no JS-side unescape function is needed. Written up because I was about to hand-roll one.

    Records the escape set (\xNN hex, \NNN decimal -- not octal, unless built
    with C_UNESCAPE_OCTAL_SUPPORT), which readers unescape (iniGetString and
    the named-string-list readers behind File.iniGetObject()), that the
    separator is whichever of = or : comes first on the line -- so dir = C:\roms
    is unaffected and the two forms mix freely -- that typed reads still split
    on : but skip the unescape, and that : preserves trailing whitespace where
    = trims it. Verified against a live install with jsexec.

    Sits beside the existing trailing-comment trap in the same section, which
    still applies: a ; comment after a string value is not stripped under
    either separator.

    Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
    --- SBBSecho 3.37-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Rob Swindell (on Debian Linux)@1:103/705 to Git commit to main/sbbs/master on Sun Aug 9 07:19:11 2026
    https://gitlab.synchro.net/main/sbbs/-/commit/850de49a4c13adc69c000667
    Modified Files:
    .claude/skills/javascript/SKILL.md
    Log Message:
    javascript skill: a modified header must be read unexpanded

    put_msg_header() refuses a header fetched with field expansion, and the
    error it gives -- "Message header has 'expanded fields'" -- describes
    the header rather than the call that produced it, so it reads as "this
    message cannot be rewritten" instead of "your read was wrong". Since
    expansion is the default, the obvious spelling of a read-modify-write
    fails and the fix is not in the failing call.

    Documents the expand argument on get_msg_header(), notes that the same
    applies to headers taken from get_all_msg_headers(), and says to leave expansion on for read-only callers -- it is what resolves the fields
    they want. Also points at setting the field at save_msg() time, which
    avoids the round trip where the original write is yours.

    Sits with the sibling traps the section already carries: remove_msg()
    silently returning false when handed a header object, and the lazy-field gotcha. All three are calls that fail for a reason the symptom points
    away from.

    Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
    --- SBBSecho 3.37-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Rob Swindell (on Debian Linux)@1:103/705 to Git commit to main/sbbs/master on Sun Aug 9 07:19:11 2026
    https://gitlab.synchro.net/main/sbbs/-/commit/3bb1dcd78f9fad924c804e57
    Modified Files:
    .claude/skills/javascript/SKILL.md
    Log Message:
    javascript skill: say what the avoided work is, not "the round trip"

    "Setting a field at save time avoids the round trip entirely" was meant
    to point at the read-modify-write recipe directly above it, but "round
    trip" reads as network latency and never names what is being skipped.

    Says it plainly instead: there is nothing to read back and rewrite, no
    expand argument to get wrong, and no half-corrected message if the
    write fails partway through a batch. Also scopes it -- the advice only
    applies where the original write is yours, which is the distinction
    that decides which of the two approaches a reader needs.

    Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
    --- SBBSecho 3.37-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)