Skip to content

feat: basic functionality - #1

Merged
lukaslihotzki-f merged 1 commit into
mainfrom
dev
Aug 4, 2026
Merged

feat: basic functionality#1
lukaslihotzki-f merged 1 commit into
mainfrom
dev

Conversation

@lukaslihotzki-f

Copy link
Copy Markdown
Collaborator

No description provided.

@lukaslihotzki-f
lukaslihotzki-f force-pushed the dev branch 5 times, most recently from 986bf6c to f508560 Compare August 3, 2026 11:13

@jannden jannden left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just two small findings. I am still surprised Fable went with pure JS though :)

Comment thread js/app.js
Comment thread js/dows.js Outdated
@lukaslihotzki-f

Copy link
Copy Markdown
Collaborator Author

I am still surprised Fable went with pure JS though :)

This was the initial thinking: "I'm starting from scratch and need to set up all the project files.I'm starting with an empty project and will build it as plain ES modules without a build step. I'll begin by creating the core protocol modules: DNS wire format encoding/parsing, TSIG signing, DoWS WebSocket transport, AXFR/UPDATE operations, and supporting utilities for zone storage, virtual lists, and rdata conversion."

The only time I mentioned a lanugage in the initial prompt was this: "Note that the list of records can be large, so do not create actual HTML elements per record. Also, keep the zone representation in JS as compressed as possible." This implies that the run time zone representation is in JS, but TS would also have a JS run time representation (because it compiles to JS), so the requirement does not prevent the use of TS. Maybe AI understood it differently, but this is how I meant it.

@jannden jannden left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lukaslihotzki-f
lukaslihotzki-f merged commit 313cef2 into main Aug 4, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants