{"action":"created","issue":{"url":"https://api.github.com/repos/CoNarrative/precept/issues/44","repository_url":"https://api.github.com/repos/CoNarrative/precept","labels_url":"https://api.github.com/repos/CoNarrative/precept/issues/44/labels{/name}","comments_url":"https://api.github.com/repos/CoNarrative/precept/issues/44/comments","events_url":"https://api.github.com/repos/CoNarrative/precept/issues/44/events","html_url":"https://github.com/CoNarrative/precept/issues/44","id":229061577,"number":44,"title":"Rule registry may not be getting written to in CLJS","user":{"login":"alex-dixon","id":9045165,"avatar_url":"https://avatars0.githubusercontent.com/u/9045165?v=3","gravatar_id":"","url":"https://api.github.com/users/alex-dixon","html_url":"https://github.com/alex-dixon","followers_url":"https://api.github.com/users/alex-dixon/followers","following_url":"https://api.github.com/users/alex-dixon/following{/other_user}","gists_url":"https://api.github.com/users/alex-dixon/gists{/gist_id}","starred_url":"https://api.github.com/users/alex-dixon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/alex-dixon/subscriptions","organizations_url":"https://api.github.com/users/alex-dixon/orgs","repos_url":"https://api.github.com/users/alex-dixon/repos","events_url":"https://api.github.com/users/alex-dixon/events{/privacy}","received_events_url":"https://api.github.com/users/alex-dixon/received_events","type":"User","site_admin":false},"labels":[{"id":555513723,"url":"https://api.github.com/repos/CoNarrative/precept/labels/bug","name":"bug","color":"C5283D","default":true}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2017-05-16T14:51:02Z","updated_at":"2017-07-13T17:14:04Z","closed_at":"2017-07-13T17:14:04Z","body":"Have seen the rules atom evaluate to `[]` in CLJS. Was on a branch with a lot of changes though. Need to verify this is the case on master and see if the same occurs in CLJ.\r\n\r\nOur only current use for the registry is internal and inessential to our implementation (evidently).  We would like to have it available for future use, particularly to `ns-unmap` all rules while developing to ensure the only rules loaded in the REPL are the ones currently defined.\r\n\r\nProblem may be that we are registering in CLJ and trying to deref the atom in CLJS. Clara uses something nearly identical to this, but I don't know whether the value of their registry is accessible from CLJS. This isn't a problem for them but if we want tooling that uses this information in CLJS I think we'd need to be able to have the values accessible from CLJS somehow."},"comment":{"url":"https://api.github.com/repos/CoNarrative/precept/issues/comments/315143113","html_url":"https://github.com/CoNarrative/precept/issues/44#issuecomment-315143113","issue_url":"https://api.github.com/repos/CoNarrative/precept/issues/44","id":315143113,"user":{"login":"alex-dixon","id":9045165,"avatar_url":"https://avatars0.githubusercontent.com/u/9045165?v=3","gravatar_id":"","url":"https://api.github.com/users/alex-dixon","html_url":"https://github.com/alex-dixon","followers_url":"https://api.github.com/users/alex-dixon/followers","following_url":"https://api.github.com/users/alex-dixon/following{/other_user}","gists_url":"https://api.github.com/users/alex-dixon/gists{/gist_id}","starred_url":"https://api.github.com/users/alex-dixon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/alex-dixon/subscriptions","organizations_url":"https://api.github.com/users/alex-dixon/orgs","repos_url":"https://api.github.com/users/alex-dixon/repos","events_url":"https://api.github.com/users/alex-dixon/events{/privacy}","received_events_url":"https://api.github.com/users/alex-dixon/received_events","type":"User","site_admin":false},"created_at":"2017-07-13T17:14:04Z","updated_at":"2017-07-13T17:14:04Z","body":"PR for #80 should address this. We have the ability to access a rule registry at compile time and runtime though these must be in separate locations for CLJS. We have `state/rules` which for CLJS we are not maintaining as well as we could. The PR for 80 adds and uses the compile-time index in env/*compiler* but we should maintain the runtime index so users can see what rules are registered. Continuing in #44."}}
{"actor":{"display_login":"alex-dixon"}}