Skip to content

Add SwiftUI iOS demo, CLI smoke tests, HTML cleanup, and CI workflow - #7

Merged
etherlink-intern merged 2 commits into
mainfrom
codex/review-repository-vyyk1r
May 31, 2026
Merged

etherlink-intern merged 2 commits into
mainfrom
codex/review-repository-vyyk1r

Conversation

@etherlink-intern

Copy link
Copy Markdown
Owner

Motivation

  • Provide a minimal SwiftUI iOS demo that exercises the SwiftMarkItDown package end-to-end from Xcode and the simulator.
  • Add reproducible CLI smoke tests and CI to validate conversions across common formats on pushes and PRs.
  • Improve HTML conversion robustness by ignoring document <head> content and normalizing line/whitespace trimming.

Description

  • Add a minimal SwiftUI demo app (App/SwiftMarkItDownApp/ContentView.swift, SwiftMarkItDownApp.swift) and Xcode project/scheme files to embed the package.
  • Add a GitHub Actions workflow (.github/workflows/ci.yml) that runs swift test, Scripts/smoke-test.sh, and an xcodebuild of the iOS demo on PRs, pushes to main, and manual dispatch.
  • Add a CLI smoke-test runner (Scripts/smoke-test.sh) plus fixtures (Tests/Fixtures/*) and expected outputs (Tests/Expected/*) exercised by the script.
  • Update Sources/SwiftMarkItDown/Converters/HTMLConverter.swift to strip <head> blocks, trim per-line whitespace, and tighten HTML->Markdown normalization.
  • Update unit tests (Tests/SwiftMarkItDownTests/SwiftMarkItDownTests.swift) and README.md to document the demo app, smoke tests, and CI.

Testing

  • Ran swift test to execute the core unit test suite and verify conversion expectations, and it succeeded.
  • Ran Scripts/smoke-test.sh to build the CLI and compare fixture outputs against Tests/Expected, and it succeeded.
  • Built the iOS demo with xcodebuild -project SwiftMarkItDownApp.xcodeproj -scheme SwiftMarkItDownApp -destination 'generic/platform=iOS Simulator' CODE_SIGNING_ALLOWED=NO build to verify the app target, and the build succeeded.

Codex Task

@etherlink-intern
etherlink-intern marked this pull request as ready for review May 31, 2026 18:07
@etherlink-intern
etherlink-intern merged commit 0edf4f6 into main May 31, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant