Autobuild onboarding: .obvious/ contract with verified local dev stack - #7
Open
obvious-autobuild-dev[bot] wants to merge 1 commit into
Open
Autobuild onboarding: .obvious/ contract with verified local dev stack#7obvious-autobuild-dev[bot] wants to merge 1 commit into
obvious-autobuild-dev[bot] wants to merge 1 commit into
Conversation
Replaces the prior .obvious/ revision (which recorded an unhealthy dev stack and no snapshot) with a fully verified environment: - dev_stack_healthy: true (Mono 6.12 + xsp4 serving on :8080) - sandbox snapshot q1o1drfyu78vckibf184:default captured - all primary flows verified end-to-end (HTTP + browser automation) - adds .obvious/skills/local-dev/SKILL.md documenting the setup
There was a problem hiding this comment.
Obvious Code Review
Verdict: COMMENT
No findings — clean pass.
Reviewed the canonical patch (3 files, +100/−35, docs-only .obvious/ onboarding contract). Every repo-facing factual claim was verified against the source tree: package inventory (14 packages, versions match the vulnerability table), Web.config flags and Roslyn provider, csproj IIS Express port, CI workflow steps, gitignore coverage, file inventories, view content, and bundle configuration. All accurate and internally consistent across the three files.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Autobuild onboarding run for this repo. The prior
.obvious/revision (already onmaster) recordeddev_stack_healthy: Noand no snapshot — this PR replaces it with a fully verified local dev environment and a sandbox snapshot.Files generated
.obvious/obvious.md— top-level agent guidance: stack, commands, local verification summary, snapshot info (81 lines).obvious/codebase-map.md— folder-level map (depth 2, single table).obvious/config.yml— repo policy: default branchmaster, merge methodsquash(auto-detected from GitHub repo settings; unchanged from prior revision).obvious/skills/local-dev/SKILL.md— durable record of the verified local dev environment (new)Source files scanned
README.md,SimpleFrameworkApp.sln,SimpleFrameworkApp.csproj,packages.config(14 packages),Web.config+Views/Web.config,Global.asax/Global.asax.cs,App_Start/{BundleConfig,FilterConfig,RouteConfig}.cs,Controllers/HomeController.cs,Views/**(6 Razor views),.github/workflows/build.yml,.gitignore. No AGENTS.md / CONTRIBUTING.md present.Local dev validation result
PASS —
dev_stack_healthy: truemono/xsp@3330ff0, installed to/usr/local, assemblies registered in the mono GAC). No external services, env vars, or secrets are required.mono ~/tools/nuget.exe restore SimpleFrameworkApp.sln— exit 0, 14 packages intopackages/. Emits a non-fatalMSBUILD0004warning (NuGet probing the deprecated xbuild shim) — documented in the skill.~/tools/build-sfa.sh—mcscompiles all 6 C# files →bin/SimpleFrameworkApp.dlland stages dependency DLLs + Roslyn compilers intobin/(MSBuild for net48 does not exist on Linux; the msbuild build runs only in CI onwindows-latest).xsp4 --port 8080 --nonstopfrom the repo root.GET /→ 200, home page rendered (<h1>ASP.NET</h1>, Razor view compiled at runtime)GET /Home/About→ 200 (<h2>About.</h2>),GET /Home/Contact→ 200 (<h2>Contact.</h2>)/Content/bootstrap.css(145933 B) and/Scripts/jquery-3.4.1.js(280364 B) → 200GET /Home/NoSuchAction→ 404 (correct MVC behavior)~/evidence/{home,about,contact,contact-after-nav}.pngin the sandbox.Sandbox snapshot
q1o1drfyu78vckibf184:default~/tools/nuget.exe), build script (~/tools/build-sfa.sh), Chromium 151 + playwright-core (~/tools/shots/), restoredpackages/, builtbin/, evidence screenshots, and the dev server running on port 8080.Blockers
None.
Explicit omissions
codebase-map.mdis a separate file rather than inlined inobvious.md— the repo has 9 top-level directories (inline is reserved for ≤2).nuget restore/msbuild/ IIS Express) are documented as canonical but were not verified in this environment; the Linux/Mono path is the verified one.