Skip to content

Production hardening pass - #1

Merged
slavazeph-coder merged 1 commit into
mainfrom
hardening/v0.1.1
Jul 2, 2026
Merged

slavazeph-coder merged 1 commit into
mainfrom
hardening/v0.1.1

Conversation

@slavazeph-coder

Copy link
Copy Markdown
Contributor

Upgrades Reconstruct to AppSpec 0.2 with stricter validation, bounded capture, integrity verification, expanded tests, npm workspaces, CI, dependency auditing, and operational documentation.

Local verification completed successfully with the unit, policy, browser integration, build, and dependency audit checks.

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 05afbe7fb8

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

"type":"object","additionalProperties":false,"required":["name","sourceUrl"],
"properties": {
"name":{"type":"string","minLength":1,"maxLength":300},
"sourceUrl":{"type":"string","format":"uri","pattern":"^https?://"},

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Reject credentialed source URLs in the schema

When downstream consumers validate AppSpecs with the exported JSON Schema, this accepts https://user:pass@example.com/... because format: uri plus the ^https?:// pattern still permits URL userinfo, while the runtime validator rejects credentials. That makes schema-valid files fail the CLI later and can preserve secrets that the runtime intentionally blocks; mirror isHttpUrl here by excluding credentialed URLs.

Useful? React with 👍 / 👎.

@slavazeph-coder
slavazeph-coder merged commit cb5139a into main Jul 2, 2026
5 of 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.

1 participant