Reorganize and clean up plugin

bin/run-plugin-tests.sh

@@ -14,13 +14,5 @@ remodel bin/put-plugin-in-test-place.lua "$PLUGIN_FILE" "$PLACE_FILE"
 run-in-roblox -s plugin/testBootstrap.server.lua "$PLACE_FILE"
 
 pushd plugin
-luacheck src
+luacheck src log http
-popd
-
-pushd rojo-plugin-http
-luacheck src
-popd
-
-pushd rojo-plugin-log
-luacheck src
 popd

bin/run-tests.sh

@@ -1,7 +0,0 @@
-#!/bin/sh
-
-set -ev
-
-cargo test --all --locked --verbose
-cargo fmt -- --check
-cargo clippy

plugin/.luacheckrc

@@ -20,6 +20,7 @@ stds.roblox = {
 		"CFrame",
 		"Enum",
 		"Instance",
+		"DockWidgetPluginGuiInfo",
 	}
 }
 

plugin/default.project.json

@@ -5,6 +5,12 @@
     "Plugin": {
       "$path": "src"
     },
+    "Log": {
+      "$path": "log"
+    },
+    "Http": {
+      "$path": "http"
+    },
     "Roact": {
       "$path": "modules/roact/src"
     },

plugin/http/Error.lua

@@ -1,9 +1,7 @@
-local Logging = require(script.Parent.Logging)
+local Error = {}
+Error.__index = Error
 
-local HttpError = {}
+Error.Kind = {
-HttpError.__index = HttpError
-
-HttpError.Error = {
 	HttpNotEnabled = {
 		message = "Rojo requires HTTP access, which is not enabled.\n" ..
 			"Check your game settings, located in the 'Home' tab of Studio.",
@@ -20,13 +18,13 @@ HttpError.Error = {
 	},
 }
 
-setmetatable(HttpError.Error, {
+setmetatable(Error.Kind, {
 	__index = function(_, key)
-		error(("%q is not a valid member of HttpError.Error"):format(tostring(key)), 2)
+		error(("%q is not a valid member of Http.Error.Kind"):format(tostring(key)), 2)
 	end,
 })
 
-function HttpError.new(type, extraMessage)
+function Error.new(type, extraMessage)
 	extraMessage = extraMessage or ""
 	local message = type.message:gsub("{{message}}", extraMessage)
 
@@ -35,38 +33,34 @@ function HttpError.new(type, extraMessage)
 		message = message,
 	}
 
-	setmetatable(err, HttpError)
+	setmetatable(err, Error)
 
 	return err
 end
 
-function HttpError:__tostring()
+function Error:__tostring()
 	return self.message
 end
 
 --[[
 	This method shouldn't have to exist. Ugh.
 ]]
-function HttpError.fromErrorString(message)
+function Error.fromErrorString(message)
 	local lower = message:lower()
 
 	if lower:find("^http requests are not enabled") then
-		return HttpError.new(HttpError.Error.HttpNotEnabled)
+		return Error.new(Error.Kind.HttpNotEnabled)
 	end
 
 	if lower:find("^httperror: timedout") then
-		return HttpError.new(HttpError.Error.Timeout)
+		return Error.new(Error.Kind.Timeout)
 	end
 
 	if lower:find("^httperror: connectfail") then
-		return HttpError.new(HttpError.Error.ConnectFailed)
+		return Error.new(Error.Kind.ConnectFailed)
 	end
 
-	return HttpError.new(HttpError.Error.Unknown, message)
+	return Error.new(Error.Kind.Unknown, message)
 end
 
-function HttpError:report()
+return Error
-	Logging.warn(self.message)
-end
-
-return HttpError

plugin/http/Response.lua

@@ -1,34 +1,34 @@
 local HttpService = game:GetService("HttpService")
 
 local stringTemplate = [[
-HttpResponse {
+Http.Response {
 	code: %d
 	body: %s
 }]]
 
-local HttpResponse = {}
+local Response = {}
-HttpResponse.__index = HttpResponse
+Response.__index = Response
 
-function HttpResponse:__tostring()
+function Response:__tostring()
 	return stringTemplate:format(self.code, self.body)
 end
 
-function HttpResponse.fromRobloxResponse(response)
+function Response.fromRobloxResponse(response)
 	local self = {
 		body = response.Body,
 		code = response.StatusCode,
 		headers = response.Headers,
 	}
 
-	return setmetatable(self, HttpResponse)
+	return setmetatable(self, Response)
 end
 
-function HttpResponse:isSuccess()
+function Response:isSuccess()
 	return self.code >= 200 and self.code < 300
 end
 
-function HttpResponse:json()
+function Response:json()
 	return HttpService:JSONDecode(self.body)
 end
 
-return HttpResponse
+return Response

plugin/http/init.lua

@@ -1,21 +1,23 @@
 local HttpService = game:GetService("HttpService")
 
-local Promise = require(script.Parent.Parent.Promise)
+local Promise = require(script.Parent.Promise)
+local Log = require(script.Parent.Log)
 
-local Logging = require(script.Parent.Logging)
+local HttpError = require(script.Error)
-local HttpError = require(script.Parent.HttpError)
+local HttpResponse = require(script.Response)
-local HttpResponse = require(script.Parent.HttpResponse)
 
 local lastRequestId = 0
 
--- TODO: Factor out into separate library, especially error handling
 local Http = {}
 
+Http.Error = HttpError
+Http.Response = HttpResponse
+
 function Http.get(url)
 	local requestId = lastRequestId + 1
 	lastRequestId = requestId
 
-	Logging.trace("GET(%d) %s", requestId, url)
+	Log.trace("GET(%d) %s", requestId, url)
 
 	return Promise.new(function(resolve, reject)
 		coroutine.wrap(function()
@@ -27,10 +29,10 @@ function Http.get(url)
 			end)
 
 			if success then
-				Logging.trace("Request %d success: status code %s", requestId, response.StatusCode)
+				Log.trace("Request %d success: status code %s", requestId, response.StatusCode)
 				resolve(HttpResponse.fromRobloxResponse(response))
 			else
-				Logging.trace("Request %d failure: %s", requestId, response)
+				Log.trace("Request %d failure: %s", requestId, response)
 				reject(HttpError.fromErrorString(response))
 			end
 		end)()
@@ -41,7 +43,7 @@ function Http.post(url, body)
 	local requestId = lastRequestId + 1
 	lastRequestId = requestId
 
-	Logging.trace("POST(%d) %s\n%s", requestId, url, body)
+	Log.trace("POST(%d) %s\n%s", requestId, url, body)
 
 	return Promise.new(function(resolve, reject)
 		coroutine.wrap(function()
@@ -54,10 +56,10 @@ function Http.post(url, body)
 			end)
 
 			if success then
-				Logging.trace("Request %d success: status code %s", requestId, response.StatusCode)
+				Log.trace("Request %d success: status code %s", requestId, response.StatusCode)
 				resolve(HttpResponse.fromRobloxResponse(response))
 			else
-				Logging.trace("Request %d failure: %s", requestId, response)
+				Log.trace("Request %d failure: %s", requestId, response)
 				reject(HttpError.fromErrorString(response))
 			end
 		end)()

plugin/http/init.spec.lua

@@ -0,0 +1,5 @@
+return function()
+	it("should load", function()
+		require(script.Parent)
+	end)
+end

plugin/log/init.lua

@@ -0,0 +1,54 @@
+local Level = {
+	Error = 0,
+	Warning = 1,
+	Info = 2,
+	Debug = 3,
+	Trace = 4,
+}
+
+local function getLogLevel()
+	return Level.Info
+end
+
+local function addTags(tag, message)
+	return tag .. message:gsub("\n", "\n" .. tag)
+end
+
+local TRACE_TAG = (" "):rep(15) .. "[Rojo-Trace] "
+local INFO_TAG = (" "):rep(15) .. "[Rojo-Info] "
+local DEBUG_TAG = (" "):rep(15) .. "[Rojo-Debug] "
+local WARN_TAG = "[Rojo-Warn] "
+
+local Log = {}
+
+Log.Level = Level
+
+function Log.setLogLevelThunk(thunk)
+	getLogLevel = thunk
+end
+
+function Log.trace(template, ...)
+	if getLogLevel() >= Level.Trace then
+		print(addTags(TRACE_TAG, string.format(template, ...)))
+	end
+end
+
+function Log.info(template, ...)
+	if getLogLevel() >= Level.Info then
+		print(addTags(INFO_TAG, string.format(template, ...)))
+	end
+end
+
+function Log.debug(template, ...)
+	if getLogLevel() >= Level.Debug then
+		print(addTags(DEBUG_TAG, string.format(template, ...)))
+	end
+end
+
+function Log.warn(template, ...)
+	if getLogLevel() >= Level.Warning then
+		warn(addTags(WARN_TAG, string.format(template, ...)))
+	end
+end
+
+return Log

plugin/log/init.spec.lua

@@ -0,0 +1,5 @@
+return function()
+	it("should load", function()
+		require(script.Parent)
+	end)
+end

plugin/src/ApiContext.lua

@@ -1,9 +1,8 @@
 local Promise = require(script.Parent.Parent.Promise)
+local Http = require(script.Parent.Parent.Http)
 
 local Config = require(script.Parent.Config)
 local Version = require(script.Parent.Version)
-local Http = require(script.Parent.Http)
-local HttpError = require(script.Parent.HttpError)
 
 local ApiContext = {}
 ApiContext.__index = ApiContext
@@ -143,7 +142,7 @@ function ApiContext:retrieveMessages()
 	local function sendRequest()
 		return Http.get(url)
 			:catch(function(err)
-				if err.type == HttpError.Error.Timeout then
+				if err.type == Http.Error.Kind.Timeout then
 					return sendRequest()
 				end
 

plugin/src/Components/RojoFooter.lua

@@ -8,8 +8,6 @@ local Version = require(Plugin.Version)
 local Assets = require(Plugin.Assets)
 local Theme = require(Plugin.Theme)
 
-local FitText = require(Plugin.Components.FitText)
-
 local e = Roact.createElement
 
 local RojoFooter = Roact.Component:extend("RojoFooter")

plugin/src/Logging.lua

@@ -1,50 +1,2 @@
-local DevSettings = require(script.Parent.DevSettings)
+-- TODO: Remove this once all references have been updated.
-
+return require(script.Parent.Parent.Log)
-local Level = {
-	Error = 0,
-	Warning = 1,
-	Info = 2,
-	Trace = 3,
-}
-
-local testLogLevel = nil
-
-local function getLogLevel()
-	if testLogLevel ~= nil then
-		return testLogLevel
-	end
-
-	return DevSettings:getLogLevel()
-end
-
-local function addTags(tag, message)
-	return tag .. message:gsub("\n", "\n" .. tag)
-end
-
-local INFO_TAG = (" "):rep(15) .. "[Rojo-Info] "
-local TRACE_TAG = (" "):rep(15) .. "[Rojo-Trace] "
-local WARN_TAG = "[Rojo-Warn] "
-
-local Log = {}
-
-Log.Level = Level
-
-function Log.trace(template, ...)
-	if getLogLevel() >= Level.Trace then
-		print(addTags(TRACE_TAG, string.format(template, ...)))
-	end
-end
-
-function Log.info(template, ...)
-	if getLogLevel() >= Level.Info then
-		print(addTags(INFO_TAG, string.format(template, ...)))
-	end
-end
-
-function Log.warn(template, ...)
-	if getLogLevel() >= Level.Warning then
-		warn(addTags(WARN_TAG, string.format(template, ...)))
-	end
-end
-
-return Log

plugin/src/Reconciler.lua

@@ -161,7 +161,10 @@ function Reconciler:__reify(virtualInstancesById, id, parent)
 	end)
 
 	if not ok then
-		error(("Couldn't create an Instance of type %q, a child of %s"):format(virtualInstance.ClassName, parent:GetFullName()))
+		error(("Couldn't create an Instance of type %q, a child of %s"):format(
+			virtualInstance.ClassName,
+			parent:GetFullName()
+		))
 	end
 
 	for key, value in pairs(virtualInstance.Properties) do

plugin/src/init.server.lua

@@ -2,6 +2,14 @@ if not plugin then
 	return
 end
 
+local Log = require(script.Parent.Log)
+
+local DevSettings = require(script.DevSettings)
+
+Log.setLogLevelThunk(function()
+	return DevSettings:getLogLevel()
+end)
+
 local Roact = require(script.Parent.Roact)
 
 local Config = require(script.Config)

plugin/testBootstrap.server.lua

@@ -12,7 +12,7 @@ if setDevSettings then
 	DevSettings:createTestSettings()
 end
 
-TestEZ.TestBootstrap:run({Rojo.Plugin})
+TestEZ.TestBootstrap:run({ Rojo.Plugin, Rojo.Http, Rojo.Log })
 
 if setDevSettings then
 	DevSettings:resetValues()