From b79a73d62dcc346a1df674477d839a9ac1e340f2 Mon Sep 17 00:00:00 2001 From: Zsolt Tasnadi Date: Tue, 4 Aug 2026 19:09:11 +0200 Subject: [PATCH] Phase 4: move catalog controllers and routes into WarpEngine - update, files and the 6 /api catalog controllers now live in the engine on a new WarpEngine::ApiController base (same rescue/mime behavior as host) - engine routes serve /update, /file/*path, /api/software*, /api/builds*, /api/image/:id, /api/download at unchanged public paths via the root mount; host routes keep only TTG endpoints (events, members, wiki, rss, swagger) - /update secret comes from WarpEngine.config.update_secret and an unconfigured secret now rejects every request (previously an empty UPDATE_SECRET env accepted empty secrets) - apipie-rails is an engine dependency (DSL in engine controllers); dummy app configures apipie with validation off, mirroring the host - engine request specs: catalog controller specs moved from host plus new /update auth contract spec Verified: engine suite 53 green, host suite 6 green, /api/software and /api/builds byte-identical to baselines, /update 401/400 behavior intact, admin and TTG endpoints OK. Co-Authored-By: Claude Fable 5 --- apps/api/Gemfile.lock | 1 + .../app/controllers/api/builds_controller.rb | 5 -- .../controllers/api/downloads_controller.rb | 29 --------- .../app/controllers/api/images_controller.rb | 15 ----- .../api/software_builds_controller.rb | 5 -- .../controllers/api/software_controller.rb | 59 ------------------ .../api/software_highlighted_controller.rb | 43 ------------- apps/api/app/controllers/files_controller.rb | 20 ------ apps/api/app/controllers/update_controller.rb | 45 -------------- apps/api/config/routes.rb | 12 +--- .../warp_engine/api/builds_controller.rb | 7 +++ .../warp_engine/api/downloads_controller.rb | 31 ++++++++++ .../warp_engine/api/images_controller.rb | 17 ++++++ .../api/software_builds_controller.rb | 7 +++ .../warp_engine/api/software_controller.rb | 61 +++++++++++++++++++ .../api/software_highlighted_controller.rb | 45 ++++++++++++++ .../controllers/warp_engine/api_controller.rb | 32 ++++++++++ .../warp_engine/files_controller.rb | 22 +++++++ .../warp_engine/update_controller.rb | 49 +++++++++++++++ libs/ruby/warp_engine/config/routes.rb | 11 ++++ libs/ruby/warp_engine/lib/warp_engine.rb | 1 + .../spec/dummy/config/initializers/apipie.rb | 11 ++++ .../spec/requests}/builds_controller_spec.rb | 2 +- .../requests}/downloads_controller_spec.rb | 2 +- .../software_builds_controller_spec.rb | 2 +- .../requests}/software_controller_spec.rb | 2 +- .../spec/requests/update_controller_spec.rb | 46 ++++++++++++++ libs/ruby/warp_engine/warp_engine.gemspec | 1 + 28 files changed, 349 insertions(+), 234 deletions(-) delete mode 100644 apps/api/app/controllers/api/builds_controller.rb delete mode 100644 apps/api/app/controllers/api/downloads_controller.rb delete mode 100644 apps/api/app/controllers/api/images_controller.rb delete mode 100644 apps/api/app/controllers/api/software_builds_controller.rb delete mode 100644 apps/api/app/controllers/api/software_controller.rb delete mode 100644 apps/api/app/controllers/api/software_highlighted_controller.rb delete mode 100644 apps/api/app/controllers/files_controller.rb delete mode 100644 apps/api/app/controllers/update_controller.rb create mode 100644 libs/ruby/warp_engine/app/controllers/warp_engine/api/builds_controller.rb create mode 100644 libs/ruby/warp_engine/app/controllers/warp_engine/api/downloads_controller.rb create mode 100644 libs/ruby/warp_engine/app/controllers/warp_engine/api/images_controller.rb create mode 100644 libs/ruby/warp_engine/app/controllers/warp_engine/api/software_builds_controller.rb create mode 100644 libs/ruby/warp_engine/app/controllers/warp_engine/api/software_controller.rb create mode 100644 libs/ruby/warp_engine/app/controllers/warp_engine/api/software_highlighted_controller.rb create mode 100644 libs/ruby/warp_engine/app/controllers/warp_engine/api_controller.rb create mode 100644 libs/ruby/warp_engine/app/controllers/warp_engine/files_controller.rb create mode 100644 libs/ruby/warp_engine/app/controllers/warp_engine/update_controller.rb create mode 100644 libs/ruby/warp_engine/spec/dummy/config/initializers/apipie.rb rename {apps/api/spec/controllers => libs/ruby/warp_engine/spec/requests}/builds_controller_spec.rb (89%) rename {apps/api/spec/controllers => libs/ruby/warp_engine/spec/requests}/downloads_controller_spec.rb (90%) rename {apps/api/spec/controllers => libs/ruby/warp_engine/spec/requests}/software_builds_controller_spec.rb (93%) rename {apps/api/spec/controllers => libs/ruby/warp_engine/spec/requests}/software_controller_spec.rb (95%) create mode 100644 libs/ruby/warp_engine/spec/requests/update_controller_spec.rb diff --git a/apps/api/Gemfile.lock b/apps/api/Gemfile.lock index 8cb05d2..c37b62d 100644 --- a/apps/api/Gemfile.lock +++ b/apps/api/Gemfile.lock @@ -2,6 +2,7 @@ PATH remote: ../libs/ruby/warp_engine specs: warp_engine (0.1.0) + apipie-rails blueprinter rails (>= 8.0) rubyzip (~> 2.3) diff --git a/apps/api/app/controllers/api/builds_controller.rb b/apps/api/app/controllers/api/builds_controller.rb deleted file mode 100644 index b19ded8..0000000 --- a/apps/api/app/controllers/api/builds_controller.rb +++ /dev/null @@ -1,5 +0,0 @@ -class Api::BuildsController < ApiController - def index - render json: WarpEngine::BuildsService.new.index - end -end diff --git a/apps/api/app/controllers/api/downloads_controller.rb b/apps/api/app/controllers/api/downloads_controller.rb deleted file mode 100644 index b7d61d0..0000000 --- a/apps/api/app/controllers/api/downloads_controller.rb +++ /dev/null @@ -1,29 +0,0 @@ -class Api::DownloadsController < ApiController - resource_description do - short "File downloads" - formats [ "binary" ] - end - - api :GET, "/api/download", "Download a file by path" - param :path, String, required: true, desc: "File path to download" - returns code: 200, desc: "File binary data" - error code: 400, desc: "Path is blank" - error code: 404, desc: "File not found" - def show - path = params[:path] - return render(json: { error: "Path is required" }, status: :bad_request) if path.blank? - - full_path = WarpEngine::DownloadService.new.create( - path: path, - ip: request.remote_ip, - user_agent: request.user_agent, - referer: request.referer - ) - - if full_path - send_file full_path, disposition: "attachment", type: resolve_mime(full_path) - else - render json: { error: "Not found" }, status: :not_found - end - end -end diff --git a/apps/api/app/controllers/api/images_controller.rb b/apps/api/app/controllers/api/images_controller.rb deleted file mode 100644 index 39a84b8..0000000 --- a/apps/api/app/controllers/api/images_controller.rb +++ /dev/null @@ -1,15 +0,0 @@ -class Api::ImagesController < ApiController - resource_description do - short "Images" - formats [ "binary" ] - end - - api :GET, "/api/image/:id", "Get image by ID" - param :id, :number, required: true, desc: "Image ID" - returns code: 200, desc: "Image binary data" - error code: 404, desc: "Image not found" - def show - image = WarpEngine::ImageService.new.show(WarpEngine::ImageShowInputDto.new(id: params[:id])) - send_file image.file_path, type: image.content_type, disposition: "inline" - end -end diff --git a/apps/api/app/controllers/api/software_builds_controller.rb b/apps/api/app/controllers/api/software_builds_controller.rb deleted file mode 100644 index f92abbe..0000000 --- a/apps/api/app/controllers/api/software_builds_controller.rb +++ /dev/null @@ -1,5 +0,0 @@ -class Api::SoftwareBuildsController < ApiController - def show - render json: WarpEngine::BuildsService.new.show(params[:name]) - end -end diff --git a/apps/api/app/controllers/api/software_controller.rb b/apps/api/app/controllers/api/software_controller.rb deleted file mode 100644 index c64e696..0000000 --- a/apps/api/app/controllers/api/software_controller.rb +++ /dev/null @@ -1,59 +0,0 @@ -class Api::SoftwareController < ApiController - resource_description do - short "Software catalog" - end - - def_param_group :external_link do - property :ID, Integer, desc: "Link ID" - property :softwareId, Integer, desc: "Parent software ID" - property :label, String, desc: "Link label (e.g. GitHub)" - property :url, String, desc: "Link URL" - end - - def_param_group :software_image do - property :url, String, desc: "Image URL (e.g. /api/image/123)" - property :isDefault, :boolean, desc: "Default image flag" - property :position, Integer, desc: "Display order" - end - - def_param_group :release do - property :ID, Integer, desc: "Release ID" - property :softwareId, Integer, desc: "Parent software ID" - property :version, String, desc: "Version string" - property :cartridgePath, String, desc: "Cartridge file path" - property :sourcePath, String, desc: "Source file path" - property :htmlFolderPath, String, desc: "HTML playable folder path" - property :docsFolderPath, String, desc: "Documentation folder path" - property :downloadCount, Integer, desc: "Download count for this release" - end - - api :GET, "/api/software", "List all software entries with releases" - returns code: 200, desc: "Wrapper object with softwares array" do - property :softwares, Array, desc: "Array of software entries" do - property :ID, Integer, desc: "Software ID" - property :name, String, desc: "Internal name" - property :title, String, desc: "Display title" - property :author, String, desc: "Author name" - property :desc, String, desc: "Short description" - property :story, String, desc: "Long description / story" - property :license, String, desc: "License type" - property :platform, String, desc: "Platform (tic80, love, ebitengine, c64, godot, bevy, phaser)" - property :status, String, desc: "Status (active, inactive)" - property :highlighted, :boolean, desc: "Currently highlighted" - property :imageUrl, String, desc: "Default image URL" - property :externalLinks, Array, desc: "External links" do - property :ID, Integer, desc: "Link ID" - property :label, String, desc: "Link label" - property :url, String, desc: "Link URL" - end - property :images, Array, desc: "Image gallery" do - property :url, String, desc: "Image URL" - property :isDefault, :boolean, desc: "Default image flag" - property :position, Integer, desc: "Display order" - end - end - end - def index - render json: WarpEngine::SoftwareService.new.index - end -end diff --git a/apps/api/app/controllers/api/software_highlighted_controller.rb b/apps/api/app/controllers/api/software_highlighted_controller.rb deleted file mode 100644 index 5516670..0000000 --- a/apps/api/app/controllers/api/software_highlighted_controller.rb +++ /dev/null @@ -1,43 +0,0 @@ -class Api::SoftwareHighlightedController < ApiController - resource_description do - short "Highlighted software" - end - - api :GET, "/api/software/highlighted", "Get currently highlighted software entry" - returns code: 200, desc: "Highlighted software with releases and stats" do - property :software, Hash, desc: "Software entry" do - property :ID, Integer, desc: "Software ID" - property :name, String, desc: "Internal name" - property :title, String, desc: "Display title" - property :author, String, desc: "Author name" - property :desc, String, desc: "Short description" - property :story, String, desc: "Long description / story" - property :license, String, desc: "License type" - property :platform, String, desc: "Platform (tic80, love, ebitengine, c64, godot, bevy, phaser)" - property :status, String, desc: "Status" - property :highlighted, :boolean, desc: "Highlighted flag" - property :imageUrl, String, desc: "Default image URL" - end - property :releases, Array, desc: "All releases" do - property :ID, Integer, desc: "Release ID" - property :version, String, desc: "Version string" - property :cartridgePath, String, desc: "Cartridge path" - property :sourcePath, String, desc: "Source path" - property :htmlFolderPath, String, desc: "HTML folder path" - property :docsFolderPath, String, desc: "Docs folder path" - property :downloadCount, Integer, desc: "Download count" - end - property :latestRelease, Hash, desc: "Latest release object" - property :webPlayableRelease, Hash, desc: "Web-playable release (if any)" - property :totalDownloads, Integer, desc: "Total download count across all releases" - end - error code: 404, desc: "No highlighted software found" - def index - result = WarpEngine::SoftwareHighlightedService.new.index - if result - render json: result - else - render json: { error: "no highlighted software found" }, status: :not_found - end - end -end diff --git a/apps/api/app/controllers/files_controller.rb b/apps/api/app/controllers/files_controller.rb deleted file mode 100644 index 11f01e9..0000000 --- a/apps/api/app/controllers/files_controller.rb +++ /dev/null @@ -1,20 +0,0 @@ -class FilesController < ApiController - resource_description do - short "Static files" - formats [ "binary" ] - end - - api :GET, "/file/*path", "Serve or redirect to a file" - param :path, String, required: true, desc: "File path" - returns code: 200, desc: "File binary data" - returns code: 301, desc: "Redirect to file URL" - error code: 404, desc: "File not found" - def show - result = WarpEngine::FileService.new.show(WarpEngine::FileShowInputDto.new(path: params[:path])) - case result.type - when :redirect then redirect_to result.url, status: :moved_permanently - when :file then send_file result.path, disposition: "inline", type: resolve_mime(result.path) - when :not_found then head :not_found - end - end -end diff --git a/apps/api/app/controllers/update_controller.rb b/apps/api/app/controllers/update_controller.rb deleted file mode 100644 index 985ffec..0000000 --- a/apps/api/app/controllers/update_controller.rb +++ /dev/null @@ -1,45 +0,0 @@ -class UpdateController < ApiController - resource_description do - short "Software updater" - formats [ "text" ] - end - - rescue_from ArgumentError do |e| - render plain: e.message, status: :bad_request - end - - rescue_from StandardError do |e| - Rails.logger.error("[UpdateController] #{e.class}: #{e.message}\n#{e.backtrace.first(5).join("\n")}") - render plain: "Internal server error", status: :internal_server_error - end - - api :GET, "/update", "Update software version in database" - param :secret, String, required: true, desc: "Authorization secret" - param :platform, String, required: false, desc: "Platform (tic80, love, ebitengine, c64, godot, bevy, phaser)" - param :name, String, required: false, desc: "Software name" - param :version, String, required: true, desc: "Version string" - returns code: 200, desc: "Plain text 'Updated'" - error code: 401, desc: "Invalid secret" - error code: 400, desc: "Version not provided or invalid arguments" - error code: 500, desc: "Internal server error" - def update - return render plain: "Unauthorized", status: :unauthorized unless authorized? - return render plain: "Version not provided", status: :bad_request if params[:version].blank? - - input = WarpEngine::UpdateInputDto.new( - platform: params[:platform], - name: params[:name], - version: params[:version] - ) - - WarpEngine::UpdateService.new.update(input) - render plain: "Updated" - end - - private - - def authorized? - secret = request.headers["X-Update-Secret"].presence || params[:secret] - secret == ENV.fetch("UPDATE_SECRET", "") - end -end diff --git a/apps/api/config/routes.rb b/apps/api/config/routes.rb index 4cf121e..4789d76 100644 --- a/apps/api/config/routes.rb +++ b/apps/api/config/routes.rb @@ -5,22 +5,16 @@ Rails.application.routes.draw do namespace :api do get "swagger", to: "swagger#index" - get "software", to: "software#index" - get "software/highlighted", to: "software_highlighted#index" get "events", to: "events#index" get "members", to: "members#index" - get "image/:id", to: "images#show" get "wiki/pages", to: "wiki#index" get "rss/blog", to: "rss#blog" get "rss/releases", to: "rss#releases" get "rss/howtos", to: "rss#howtos" - get "download", to: "downloads#show" - get "builds", to: "builds#index" - get "softwares/:name/builds", to: "software_builds#show" end - get "update", to: "update#update" - get "file/*path", to: "files#show", format: false - # Utolsó sor: a host route-jai nyernek, minden mást az engine kap el. + # Utolsó sor: a host route-jai nyernek, a katalógus-útvonalakat + # (/api/software*, /api/builds*, /api/image, /api/download, /update, /file/*) + # az engine adja. mount WarpEngine::Engine => "/" end diff --git a/libs/ruby/warp_engine/app/controllers/warp_engine/api/builds_controller.rb b/libs/ruby/warp_engine/app/controllers/warp_engine/api/builds_controller.rb new file mode 100644 index 0000000..625584a --- /dev/null +++ b/libs/ruby/warp_engine/app/controllers/warp_engine/api/builds_controller.rb @@ -0,0 +1,7 @@ +module WarpEngine + class Api::BuildsController < ApiController + def index + render json: WarpEngine::BuildsService.new.index + end + end +end diff --git a/libs/ruby/warp_engine/app/controllers/warp_engine/api/downloads_controller.rb b/libs/ruby/warp_engine/app/controllers/warp_engine/api/downloads_controller.rb new file mode 100644 index 0000000..1bb070a --- /dev/null +++ b/libs/ruby/warp_engine/app/controllers/warp_engine/api/downloads_controller.rb @@ -0,0 +1,31 @@ +module WarpEngine + class Api::DownloadsController < ApiController + resource_description do + short "File downloads" + formats [ "binary" ] + end + + api :GET, "/api/download", "Download a file by path" + param :path, String, required: true, desc: "File path to download" + returns code: 200, desc: "File binary data" + error code: 400, desc: "Path is blank" + error code: 404, desc: "File not found" + def show + path = params[:path] + return render(json: { error: "Path is required" }, status: :bad_request) if path.blank? + + full_path = WarpEngine::DownloadService.new.create( + path: path, + ip: request.remote_ip, + user_agent: request.user_agent, + referer: request.referer + ) + + if full_path + send_file full_path, disposition: "attachment", type: resolve_mime(full_path) + else + render json: { error: "Not found" }, status: :not_found + end + end + end +end diff --git a/libs/ruby/warp_engine/app/controllers/warp_engine/api/images_controller.rb b/libs/ruby/warp_engine/app/controllers/warp_engine/api/images_controller.rb new file mode 100644 index 0000000..230e933 --- /dev/null +++ b/libs/ruby/warp_engine/app/controllers/warp_engine/api/images_controller.rb @@ -0,0 +1,17 @@ +module WarpEngine + class Api::ImagesController < ApiController + resource_description do + short "Images" + formats [ "binary" ] + end + + api :GET, "/api/image/:id", "Get image by ID" + param :id, :number, required: true, desc: "Image ID" + returns code: 200, desc: "Image binary data" + error code: 404, desc: "Image not found" + def show + image = WarpEngine::ImageService.new.show(WarpEngine::ImageShowInputDto.new(id: params[:id])) + send_file image.file_path, type: image.content_type, disposition: "inline" + end + end +end diff --git a/libs/ruby/warp_engine/app/controllers/warp_engine/api/software_builds_controller.rb b/libs/ruby/warp_engine/app/controllers/warp_engine/api/software_builds_controller.rb new file mode 100644 index 0000000..95b7a71 --- /dev/null +++ b/libs/ruby/warp_engine/app/controllers/warp_engine/api/software_builds_controller.rb @@ -0,0 +1,7 @@ +module WarpEngine + class Api::SoftwareBuildsController < ApiController + def show + render json: WarpEngine::BuildsService.new.show(params[:name]) + end + end +end diff --git a/libs/ruby/warp_engine/app/controllers/warp_engine/api/software_controller.rb b/libs/ruby/warp_engine/app/controllers/warp_engine/api/software_controller.rb new file mode 100644 index 0000000..3a67aeb --- /dev/null +++ b/libs/ruby/warp_engine/app/controllers/warp_engine/api/software_controller.rb @@ -0,0 +1,61 @@ +module WarpEngine + class Api::SoftwareController < ApiController + resource_description do + short "Software catalog" + end + + def_param_group :external_link do + property :ID, Integer, desc: "Link ID" + property :softwareId, Integer, desc: "Parent software ID" + property :label, String, desc: "Link label (e.g. GitHub)" + property :url, String, desc: "Link URL" + end + + def_param_group :software_image do + property :url, String, desc: "Image URL (e.g. /api/image/123)" + property :isDefault, :boolean, desc: "Default image flag" + property :position, Integer, desc: "Display order" + end + + def_param_group :release do + property :ID, Integer, desc: "Release ID" + property :softwareId, Integer, desc: "Parent software ID" + property :version, String, desc: "Version string" + property :cartridgePath, String, desc: "Cartridge file path" + property :sourcePath, String, desc: "Source file path" + property :htmlFolderPath, String, desc: "HTML playable folder path" + property :docsFolderPath, String, desc: "Documentation folder path" + property :downloadCount, Integer, desc: "Download count for this release" + end + + api :GET, "/api/software", "List all software entries with releases" + returns code: 200, desc: "Wrapper object with softwares array" do + property :softwares, Array, desc: "Array of software entries" do + property :ID, Integer, desc: "Software ID" + property :name, String, desc: "Internal name" + property :title, String, desc: "Display title" + property :author, String, desc: "Author name" + property :desc, String, desc: "Short description" + property :story, String, desc: "Long description / story" + property :license, String, desc: "License type" + property :platform, String, desc: "Platform (tic80, love, ebitengine, c64, godot, bevy, phaser)" + property :status, String, desc: "Status (active, inactive)" + property :highlighted, :boolean, desc: "Currently highlighted" + property :imageUrl, String, desc: "Default image URL" + property :externalLinks, Array, desc: "External links" do + property :ID, Integer, desc: "Link ID" + property :label, String, desc: "Link label" + property :url, String, desc: "Link URL" + end + property :images, Array, desc: "Image gallery" do + property :url, String, desc: "Image URL" + property :isDefault, :boolean, desc: "Default image flag" + property :position, Integer, desc: "Display order" + end + end + end + def index + render json: WarpEngine::SoftwareService.new.index + end + end +end diff --git a/libs/ruby/warp_engine/app/controllers/warp_engine/api/software_highlighted_controller.rb b/libs/ruby/warp_engine/app/controllers/warp_engine/api/software_highlighted_controller.rb new file mode 100644 index 0000000..9e0640f --- /dev/null +++ b/libs/ruby/warp_engine/app/controllers/warp_engine/api/software_highlighted_controller.rb @@ -0,0 +1,45 @@ +module WarpEngine + class Api::SoftwareHighlightedController < ApiController + resource_description do + short "Highlighted software" + end + + api :GET, "/api/software/highlighted", "Get currently highlighted software entry" + returns code: 200, desc: "Highlighted software with releases and stats" do + property :software, Hash, desc: "Software entry" do + property :ID, Integer, desc: "Software ID" + property :name, String, desc: "Internal name" + property :title, String, desc: "Display title" + property :author, String, desc: "Author name" + property :desc, String, desc: "Short description" + property :story, String, desc: "Long description / story" + property :license, String, desc: "License type" + property :platform, String, desc: "Platform (tic80, love, ebitengine, c64, godot, bevy, phaser)" + property :status, String, desc: "Status" + property :highlighted, :boolean, desc: "Highlighted flag" + property :imageUrl, String, desc: "Default image URL" + end + property :releases, Array, desc: "All releases" do + property :ID, Integer, desc: "Release ID" + property :version, String, desc: "Version string" + property :cartridgePath, String, desc: "Cartridge path" + property :sourcePath, String, desc: "Source path" + property :htmlFolderPath, String, desc: "HTML folder path" + property :docsFolderPath, String, desc: "Docs folder path" + property :downloadCount, Integer, desc: "Download count" + end + property :latestRelease, Hash, desc: "Latest release object" + property :webPlayableRelease, Hash, desc: "Web-playable release (if any)" + property :totalDownloads, Integer, desc: "Total download count across all releases" + end + error code: 404, desc: "No highlighted software found" + def index + result = WarpEngine::SoftwareHighlightedService.new.index + if result + render json: result + else + render json: { error: "no highlighted software found" }, status: :not_found + end + end + end +end diff --git a/libs/ruby/warp_engine/app/controllers/warp_engine/api_controller.rb b/libs/ruby/warp_engine/app/controllers/warp_engine/api_controller.rb new file mode 100644 index 0000000..1a9dce6 --- /dev/null +++ b/libs/ruby/warp_engine/app/controllers/warp_engine/api_controller.rb @@ -0,0 +1,32 @@ +module WarpEngine + class ApiController < ActionController::API + resource_description do + api_version "1.0" + formats [ "json" ] + end + + rescue_from StandardError do |e| + Rails.logger.error("[#{self.class.name}] #{e.class}: #{e.message}") + render json: { error: "Internal server error" }, status: :internal_server_error + end + + rescue_from ActiveRecord::RecordNotFound do |e| + render json: { error: "Not found" }, status: :not_found + end + + rescue_from Errno::ENOENT do |e| + render json: { error: "Not found" }, status: :not_found + end + + rescue_from ArgumentError do |e| + render json: { error: e.message }, status: :bad_request + end + + private + + def resolve_mime(path) + ext = File.extname(path.to_s).delete_prefix(".") + Mime::Type.lookup_by_extension(ext) || "application/octet-stream" + end + end +end diff --git a/libs/ruby/warp_engine/app/controllers/warp_engine/files_controller.rb b/libs/ruby/warp_engine/app/controllers/warp_engine/files_controller.rb new file mode 100644 index 0000000..17cd56d --- /dev/null +++ b/libs/ruby/warp_engine/app/controllers/warp_engine/files_controller.rb @@ -0,0 +1,22 @@ +module WarpEngine + class FilesController < ApiController + resource_description do + short "Static files" + formats [ "binary" ] + end + + api :GET, "/file/*path", "Serve or redirect to a file" + param :path, String, required: true, desc: "File path" + returns code: 200, desc: "File binary data" + returns code: 301, desc: "Redirect to file URL" + error code: 404, desc: "File not found" + def show + result = WarpEngine::FileService.new.show(WarpEngine::FileShowInputDto.new(path: params[:path])) + case result.type + when :redirect then redirect_to result.url, status: :moved_permanently + when :file then send_file result.path, disposition: "inline", type: resolve_mime(result.path) + when :not_found then head :not_found + end + end + end +end diff --git a/libs/ruby/warp_engine/app/controllers/warp_engine/update_controller.rb b/libs/ruby/warp_engine/app/controllers/warp_engine/update_controller.rb new file mode 100644 index 0000000..0f69c54 --- /dev/null +++ b/libs/ruby/warp_engine/app/controllers/warp_engine/update_controller.rb @@ -0,0 +1,49 @@ +module WarpEngine + class UpdateController < ApiController + resource_description do + short "Software updater" + formats [ "text" ] + end + + rescue_from ArgumentError do |e| + render plain: e.message, status: :bad_request + end + + rescue_from StandardError do |e| + Rails.logger.error("[UpdateController] #{e.class}: #{e.message}\n#{e.backtrace.first(5).join("\n")}") + render plain: "Internal server error", status: :internal_server_error + end + + api :GET, "/update", "Update software version in database" + param :secret, String, required: true, desc: "Authorization secret" + param :platform, String, required: false, desc: "Platform (tic80, love, ebitengine, c64, godot, bevy, phaser)" + param :name, String, required: false, desc: "Software name" + param :version, String, required: true, desc: "Version string" + returns code: 200, desc: "Plain text 'Updated'" + error code: 401, desc: "Invalid secret" + error code: 400, desc: "Version not provided or invalid arguments" + error code: 500, desc: "Internal server error" + def update + return render plain: "Unauthorized", status: :unauthorized unless authorized? + return render plain: "Version not provided", status: :bad_request if params[:version].blank? + + input = WarpEngine::UpdateInputDto.new( + platform: params[:platform], + name: params[:name], + version: params[:version] + ) + + WarpEngine::UpdateService.new.update(input) + render plain: "Updated" + end + + private + + def authorized? + secret = request.headers["X-Update-Secret"].presence || params[:secret] + expected = WarpEngine.config.update_secret + # Konfigurálatlan secret esetén az endpoint zárva marad. + expected.present? && secret == expected + end + end +end diff --git a/libs/ruby/warp_engine/config/routes.rb b/libs/ruby/warp_engine/config/routes.rb index 1935491..852dfc9 100644 --- a/libs/ruby/warp_engine/config/routes.rb +++ b/libs/ruby/warp_engine/config/routes.rb @@ -1,2 +1,13 @@ WarpEngine::Engine.routes.draw do + namespace :api do + get "software", to: "software#index" + get "software/highlighted", to: "software_highlighted#index" + get "image/:id", to: "images#show" + get "download", to: "downloads#show" + get "builds", to: "builds#index" + get "softwares/:name/builds", to: "software_builds#show" + end + + get "update", to: "update#update" + get "file/*path", to: "files#show", format: false end diff --git a/libs/ruby/warp_engine/lib/warp_engine.rb b/libs/ruby/warp_engine/lib/warp_engine.rb index 8ca83e8..0dcdd42 100644 --- a/libs/ruby/warp_engine/lib/warp_engine.rb +++ b/libs/ruby/warp_engine/lib/warp_engine.rb @@ -1,4 +1,5 @@ require "blueprinter" +require "apipie-rails" require "warp_engine/version" require "warp_engine/configuration" diff --git a/libs/ruby/warp_engine/spec/dummy/config/initializers/apipie.rb b/libs/ruby/warp_engine/spec/dummy/config/initializers/apipie.rb new file mode 100644 index 0000000..acf23d4 --- /dev/null +++ b/libs/ruby/warp_engine/spec/dummy/config/initializers/apipie.rb @@ -0,0 +1,11 @@ +Apipie.configure do |config| + config.app_name = "WarpEngine Dummy" + config.api_base_url = "" + config.doc_base_url = "/api/docs" + config.api_controllers_matcher = [ + "#{WarpEngine::Engine.root}/app/controllers/**/*.rb" + ] + config.validate = false + config.translate = false + config.default_version = "1.0" +end diff --git a/apps/api/spec/controllers/builds_controller_spec.rb b/libs/ruby/warp_engine/spec/requests/builds_controller_spec.rb similarity index 89% rename from apps/api/spec/controllers/builds_controller_spec.rb rename to libs/ruby/warp_engine/spec/requests/builds_controller_spec.rb index 57e7dcb..8eeda04 100644 --- a/apps/api/spec/controllers/builds_controller_spec.rb +++ b/libs/ruby/warp_engine/spec/requests/builds_controller_spec.rb @@ -1,6 +1,6 @@ require "rails_helper" -RSpec.describe Api::BuildsController, type: :request do +RSpec.describe "GET /api/builds", type: :request do describe "GET /api/builds" do it "returns the global build matrix" do get "/api/builds" diff --git a/apps/api/spec/controllers/downloads_controller_spec.rb b/libs/ruby/warp_engine/spec/requests/downloads_controller_spec.rb similarity index 90% rename from apps/api/spec/controllers/downloads_controller_spec.rb rename to libs/ruby/warp_engine/spec/requests/downloads_controller_spec.rb index 025391c..11e5606 100644 --- a/apps/api/spec/controllers/downloads_controller_spec.rb +++ b/libs/ruby/warp_engine/spec/requests/downloads_controller_spec.rb @@ -1,6 +1,6 @@ require "rails_helper" -RSpec.describe Api::DownloadsController, type: :request do +RSpec.describe "GET /api/download", type: :request do describe "GET /api/download" do it "returns bad_request without path" do get "/api/download" diff --git a/apps/api/spec/controllers/software_builds_controller_spec.rb b/libs/ruby/warp_engine/spec/requests/software_builds_controller_spec.rb similarity index 93% rename from apps/api/spec/controllers/software_builds_controller_spec.rb rename to libs/ruby/warp_engine/spec/requests/software_builds_controller_spec.rb index 9007b10..293f0ce 100644 --- a/apps/api/spec/controllers/software_builds_controller_spec.rb +++ b/libs/ruby/warp_engine/spec/requests/software_builds_controller_spec.rb @@ -1,6 +1,6 @@ require "rails_helper" -RSpec.describe Api::SoftwareBuildsController, type: :request do +RSpec.describe "GET /api/softwares/:name/builds", type: :request do describe "GET /api/softwares/:name/builds" do let!(:software) { create(:software, name: "test-game", platform: "love") } let!(:release) { create(:release, software: software, version: "2.0.0") } diff --git a/apps/api/spec/controllers/software_controller_spec.rb b/libs/ruby/warp_engine/spec/requests/software_controller_spec.rb similarity index 95% rename from apps/api/spec/controllers/software_controller_spec.rb rename to libs/ruby/warp_engine/spec/requests/software_controller_spec.rb index 43d9d32..fdebabe 100644 --- a/apps/api/spec/controllers/software_controller_spec.rb +++ b/libs/ruby/warp_engine/spec/requests/software_controller_spec.rb @@ -1,6 +1,6 @@ require "rails_helper" -RSpec.describe Api::SoftwareController, type: :request do +RSpec.describe "GET /api/software", type: :request do describe "GET /api/software" do it "returns all software with releases" do create(:software, name: "test-game", title: "Test Game") diff --git a/libs/ruby/warp_engine/spec/requests/update_controller_spec.rb b/libs/ruby/warp_engine/spec/requests/update_controller_spec.rb new file mode 100644 index 0000000..b623386 --- /dev/null +++ b/libs/ruby/warp_engine/spec/requests/update_controller_spec.rb @@ -0,0 +1,46 @@ +require "rails_helper" + +RSpec.describe "GET /update", type: :request do + before do + allow(WarpEngine.config).to receive(:update_secret).and_return("s3cret") + end + + it "rejects requests without a secret" do + get "/update", params: { platform: "tic80", name: "game", version: "1.0" } + + expect(response).to have_http_status(:unauthorized) + end + + it "rejects requests with a wrong secret" do + get "/update", params: { secret: "wrong", platform: "tic80", name: "game", version: "1.0" } + + expect(response).to have_http_status(:unauthorized) + end + + it "rejects every request when no secret is configured" do + allow(WarpEngine.config).to receive(:update_secret).and_return(nil) + + get "/update", params: { secret: "", platform: "tic80", name: "game", version: "1.0" } + + expect(response).to have_http_status(:unauthorized) + end + + it "requires a version" do + get "/update", headers: { "X-Update-Secret" => "s3cret" }, params: { platform: "tic80", name: "game" } + + expect(response).to have_http_status(:bad_request) + expect(response.body).to eq("Version not provided") + end + + it "runs the updater with a valid secret" do + updater = instance_double(WarpEngine::SoftwareUpdater::Tic80Service) + allow(WarpEngine::SoftwareUpdater::Tic80Service).to receive(:new).and_return(updater) + expect(updater).to receive(:update).with("game", "1.0") + + get "/update", headers: { "X-Update-Secret" => "s3cret" }, + params: { platform: "tic80", name: "game", version: "1.0" } + + expect(response).to have_http_status(:ok) + expect(response.body).to eq("Updated") + end +end diff --git a/libs/ruby/warp_engine/warp_engine.gemspec b/libs/ruby/warp_engine/warp_engine.gemspec index 2fd85a2..06a1a7c 100644 --- a/libs/ruby/warp_engine/warp_engine.gemspec +++ b/libs/ruby/warp_engine/warp_engine.gemspec @@ -20,4 +20,5 @@ Gem::Specification.new do |spec| spec.add_dependency "rails", ">= 8.0" spec.add_dependency "blueprinter" spec.add_dependency "rubyzip", "~> 2.3" + spec.add_dependency "apipie-rails" end