4.1 KiB
4.1 KiB
Created
Created by tickets.sh create.
Plan
Ticket preflight
Classification: implementation-ready after implementation Pod plan review.
Requirements are synchronized enough to ask an implementation Pod for a concrete plan, but not enough to skip planning. The Pod must plan first and wait for orchestrator approval before coding.
Current critical risks:
- Recreating Manifest authoring under another name. Profile may be manifest-like, but runtime-bound and authority-bearing fields (
pod.name, concretescope.allow, resolved paths, secret material, runtime state) must be rejected or diagnosed. - Exposing
mkManifestas the public normal API. The normal boundary should beprofile/insomnia.profile. - Implementing uncontrolled Lua
require, standard library access, or installed resource path imports. - Breaking existing profile selection semantics or resolved Manifest snapshot persistence.
- Letting Profile express capability expansion instead of scope intent/policy checked by runtime/delegation.
Current plan gate:
- Implementation Pod must produce a plan covering dependency/crate placement, sandbox, module loading, return contract, Profile data model, resolver integration, builtin/default migration, diagnostics, and tests.
- If the plan respects the boundary, the orchestrator may authorize implementation in the same Pod/worktree.
Decision
Implementation plan accepted with constraints
The implementation Pod produced a plan for Lua-based reusable Profile authoring. The plan is accepted as the implementation direction, with these constraints:
- Lua is the primary authoring path for this ticket; Nix is not the primary profile layer.
- Do not keep legacy
.nixprofile evaluation just for compatibility if it complicates the design. Prefer removing/disabling Nix profile resolution from the normal profile selector path, while leaving--manifestas the explicit low-level escape hatch. - Builtin/default must resolve from Lua/Profile or Rust in-process data without external
nix. - Public authoring boundary is
profile/require("insomnia.profile"), not publicmkManifest. - Controlled
requireis part of the implementation: host virtualinsomnia.*modules and profile-local modules only; no installed resource path imports. - Profile may be manifest-like, but runtime-bound or authority-bearing fields such as
pod.name, concretescope.allow/scope.deny, resolved paths, sockets, runtime state, and raw secret material must be rejected or clearly diagnosed. - Scope in Profile is intent/policy only; concrete authority is resolved against runtime/delegation inputs.
- Model/context-derived compaction can use Lua locals and/or helper policy such as
compact.ratio, not Nix recursive sets. - Preserve profile selection semantics where still meaningful: default/builtin/user/project/source-qualified/path selectors and persisted resolved Manifest snapshots.
Implementation plan summary:
- Add embedded Lua evaluation in
crates/manifestusing a vendored Lua crate such asmluaif dependency/license/build characteristics are acceptable. - Add a Profile data model that is a reusable manifest-like recipe template and converts into a concrete
PodManifestonly through resolver runtime inputs. - Add sandboxed Lua evaluation with denied
os,io,debug, unrestrictedpackage,dofile,loadfile, and uncontrolled loaders. - Add host-provided virtual modules such as
insomnia,insomnia.profile,insomnia.models,insomnia.compact, andinsomnia.scope. - Add profile-local controlled
requirewith canonical path checks, module cache, and cycle diagnostics. - Migrate builtin/default from
resources/nix/profiles/default.nixto a Lua/Profile source or in-process equivalent. - Add focused tests for builtin/default without external nix, host modules, local require, sandbox denial, invalid Manifest-shaped returns, scope intent resolution, and selector semantics.